You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2020/12/01 12:25:45 UTC

[GitHub] [flink] tillrohrmann commented on pull request #14028: [FLINK-20020][client] Make UnsuccessfulExecutionException part of the JobClient.getJobExecutionResult() contract

tillrohrmann commented on pull request #14028:
URL: https://github.com/apache/flink/pull/14028#issuecomment-736520139


   Thanks for the update @SteNicholas. Before moving forward I would like to hear from someone involved in the public APIs, that the `ProgramInvocationException` is indeed not public. Moreover, I still believe that the `JobExecutionException` which now transports the final `ApplicationStatus` shouldn't be used within the runtime components. Instead it should be used to communicate a failed `ApplicationStatus` to the outside world.


----------------------------------------------------------------
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:
users@infra.apache.org