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


   To expedite the merge a bit, how about just (1) switching to 
`DataOutputStream` (that is a JDK class, not a Flink class, I assume you 
searched for it in the Flink codebase?) and (2) doing the fatal error handler?
   
   I would be up for taking a patch to make failure handling more fine grained 
as a next step (it is not a strict must-have, just a nice-to-have).


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