yuchen-ecnu commented on code in PR #25798: URL: https://github.com/apache/flink/pull/25798#discussion_r1894597751
########## flink-runtime/src/main/java/org/apache/flink/streaming/api/graph/StreamGraphContext.java: ########## @@ -62,4 +62,13 @@ public interface StreamGraphContext { * @return true if all modifications were successful and applied atomically, false otherwise. */ boolean modifyStreamEdge(List<StreamEdgeUpdateRequestInfo> requestInfos); + + interface StreamGraphUpdateListener { + /** + * This method is called whenever the StreamGraph is updated. + * + * @param streamGraph the updated StreamGraph Review Comment: Thanks for your comment. As discussed offline with @JunRuiLee, the AdaptiveGraphManager has complete informations of the StreamGraph, we only need to notify the listener that the StreamGraph has been changed, so I removed the streamGraph parameter here. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org