StephanEwen commented on pull request #14104:
URL: https://github.com/apache/flink/pull/14104#issuecomment-731779775


   I think parts of this PR conflict with the changes done in #14030 
   
   Specifically with this change: 
https://github.com/apache/flink/pull/14030/files#diff-e23091be102a9049b46e9ad507bf6e65ee49d60947d5820790c7f280c7fcd8ceR101
   
   If I read this correctly, the #14030 change makes this PR here redundant, 
because it handles all exceptions already.
   
   @becketqin could you check if this makes sense?


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