Skip to content

Commit ea28cad

Browse files
committed
[hotfix] Do not reuse StreamGraph in AdaptiveGraphManagerTest when comparing the results of the adaptive graph generator and the static graph generator.
1 parent 5dd6497 commit ea28cad

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

flink-streaming-java/src/test/java/org/apache/flink/streaming/api/graph/AdaptiveGraphManagerTest.java

+7-3
Original file line numberDiff line numberDiff line change
@@ -181,9 +181,13 @@ public Tuple2<String, String> map(
181181
});
182182

183183
result.sinkTo(new DiscardingSink<>());
184-
StreamGraph streamGraph = env.getStreamGraph();
185-
JobGraph jobGraph1 = generateJobGraphInLazilyMode(streamGraph);
186-
JobGraph jobGraph2 = StreamingJobGraphGenerator.createJobGraph(streamGraph);
184+
StreamGraph streamGraph1 = env.getStreamGraph(false);
185+
JobGraph jobGraph1 = generateJobGraphInLazilyMode(streamGraph1);
186+
187+
// we could not reuse the streamGraph1 because the streamGraph1 could have been modified in
188+
// the adaptive graph manager.
189+
StreamGraph streamGraph2 = env.getStreamGraph(false);
190+
JobGraph jobGraph2 = StreamingJobGraphGenerator.createJobGraph(streamGraph2);
187191
assertThat(isJobGraphEquivalent(jobGraph1, jobGraph2)).isEqualTo(true);
188192
}
189193

0 commit comments

Comments
 (0)