You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In #317 , a PipelineExecutor instance keeps track of a PipelineStage which is reported to the client. The PipelineStage is updated and used not only in PipelineExecutor, but also in the GRPCHandler (instantiated per PipelineExecutor) and relevant code that are part of the pipeline logic.
Currently, the GRPCHandler and relevant code has no access to the PipelineStage stored at the PipelineExecutor. However, the GRPCHandler and relevant code also reports a pipeline stage to the client. This pipeline stage is hardcoded.
We would like to refactor the code so that
the PipelineStage, if reported to the client, is always bookkept at the PipelineExecutor instead of hardcoded.
logic about the PipelineStage is not scattered in multiple places.
The text was updated successfully, but these errors were encountered:
Partially addressed: We now maintain the PipelineStage centrally through PipelineExecutor decorators. However, ambiguity still exists in the GRPC client interface. I'm not sure how much value it provides to also fix this.
In #317 , a PipelineExecutor instance keeps track of a PipelineStage which is reported to the client. The PipelineStage is updated and used not only in PipelineExecutor, but also in the GRPCHandler (instantiated per PipelineExecutor) and relevant code that are part of the pipeline logic.
Currently, the GRPCHandler and relevant code has no access to the PipelineStage stored at the PipelineExecutor. However, the GRPCHandler and relevant code also reports a pipeline stage to the client. This pipeline stage is hardcoded.
We would like to refactor the code so that
The text was updated successfully, but these errors were encountered: