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


   > @SteNicholas I had a look at this solution and I would like to think on 
how to make the `status` `final`. This means that I am trying to see if we can 
have all the `JobExecutionExceptions` with a `status`. I will get back with 
more comments tomorrow.
   
   As you mentioned, all the `JobExecutionExceptions` could be with a given 
`status`. I have tried to make `status` final and updated 
`JobCancellationException`, `JobInitializationException` and 
`JobSubmissionException`. IMO, the `status` of `JobInitializationException` and 
`JobSubmissionException` is UNKNOWN, and the `status` of 
`JobCancellationException` is CANCELLED. Please check this.


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


Reply via email to