zhijiangW commented on pull request #13595:
URL: https://github.com/apache/flink/pull/13595#issuecomment-719467007


   Thanks for the updates @wsry ! 
   
   I think most of my previous comments were addressed except for 
https://github.com/apache/flink/pull/13595#discussion_r514988635. And as 
@gaoyunhaii also mentioned above, it is better to supplement some tests for 
covering empty subpartition case.


----------------------------------------------------------------
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:
[email protected]


Reply via email to