[GitHub] flink issue #5421: [FLINK-8573] [client] Add more information for printing J...

2018-05-16 Thread azagrebin 
Github user azagrebin commented on the issue:

https://github.com/apache/flink/pull/5421
  
@zhangminglei 

what do you think about creation one more `ProgramInvocationException` 
overloaded constructor with JobId as parameter? It could always append to the 
message like `MESSAGE + (Job Id is ...)` and be used wherever 
`ProgramInvocationException` is instantiated and JobId is known.

Can you please rebase on the current master to repeat the CI test?


---

[GitHub] flink issue #5421: [FLINK-8573] [client] Add more information for printing J...

2018-03-29 Thread zhangminglei 
Github user zhangminglei commented on the issue:

https://github.com/apache/flink/pull/5421
  
Sup ?


---

[GitHub] flink issue #5421: [FLINK-8573] [client] Add more information for printing J...

2018-02-09 Thread zhangminglei 
Github user zhangminglei commented on the issue:

https://github.com/apache/flink/pull/5421
  
The CI error does not relevant to this PR. 
```BackPressureStatsTrackerITCase``` test failed.


---