SteNicholas commented on pull request #14028: URL: https://github.com/apache/flink/pull/14028#issuecomment-725983640
> Hi @SteNicholas , I have a commit with my comments here https://github.com/apache/flink/tree/client-exception. Also for the tests I think we should just add tests for all the cases in the `JobResultTest`. What do you think? @kl0u Your implementation is better for me than my bad implementation. And the test cases look clear for each client. ---------------------------------------------------------------- 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]
