kl0u commented on a change in pull request #13911:
URL: https://github.com/apache/flink/pull/13911#discussion_r518597867



##########
File path: 
flink-clients/src/main/java/org/apache/flink/client/deployment/application/UnsuccessfulTerminationException.java
##########
@@ -32,11 +32,11 @@
  * application with a given {@link ApplicationStatus}.
  */
 @Internal
-public class ApplicationFailureException extends JobExecutionException {
+public class UnsuccessfulTerminationException extends JobExecutionException {

Review comment:
       Yes, I can rename it to `UnsuccessfulExecutionException`. And yes, the 
cluster is going to be shutdown one way or another.




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