Skip to content

Commit c73f177

Browse files
eliasyaoycloloxwg
authored andcommitted
fix cross meta-pipeline dependencies.
1 parent d2ed315 commit c73f177

1 file changed

Lines changed: 16 additions & 11 deletions

File tree

src/executor/executor_graph.rs

Lines changed: 16 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -200,17 +200,22 @@ impl ExecutorGraph {
200200
}
201201
}
202202

203-
// // Set up the dependencies across `MetaPipeline`.
204-
// for pipe in all_pipelines.iter() {
205-
// for pipe_ix in pipe.get_dependencies().iter() {
206-
// let from = *pipeline_graph_mapping.get(pipe_ix).unwrap();
207-
// let to =
208-
// *pipeline_graph_mapping.get(&pipe.get_pipeline_id()).unwrap();
209-
// if !graph.contains_edge(from, to) {
210-
// graph.add_edge(from, to, ());
211-
// }
212-
// }
213-
// }
203+
// Set up the dependencies across `MetaPipeline`.
204+
for pipe in all_pipelines.iter() {
205+
for pipe_ix in pipe.get_dependencies().iter() {
206+
let from = pipeline_graph_mapping
207+
.get(pipe_ix)
208+
.unwrap()
209+
.pipeline_complete_event;
210+
let to = pipeline_graph_mapping
211+
.get(&pipe.get_pipeline_id())
212+
.unwrap()
213+
.pipeline_event;
214+
if !graph.contains_edge(from, to) {
215+
graph.add_edge(from, to, ());
216+
}
217+
}
218+
}
214219
Ok(())
215220
}
216221

0 commit comments

Comments
 (0)