sjwiesman opened a new pull request #13893:
URL: https://github.com/apache/flink/pull/13893


   ## Brief changelog
   
   The underlying fetcher iterator already cancels the job on close, and 
properly handles the error case of the job already being completed, so we do 
not need to do it twice. 
   
   I would actual drop `ClientWithIterator` entirly if I didn't need it for 
backwards compat with DataStreamUtils
   
   ## Verifying this change
   
   Basic UT and also checked it manually. 
   
   


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