Thesharing commented on pull request #15314:
URL: https://github.com/apache/flink/pull/15314#issuecomment-807912254


   > I would suggest to introduce the assumption that one 
`IntermediateResultPartition` can have one only 
`ConsumerVertexGroup`(indicating that one `IntermediateDataSet` can have one 
only consumer `JobEdge` which is already a widely assumption in flink-runtime 
at the moment). This can help to reduce the complexity for each 
`vertexFinished()` invocation to O(1).
   
   Thanks for proposing this solution 👍  I've already add the check and 
comments according to this assumption. Would you mind re-reviewing it once you 
got any free time? 


-- 
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to