[GitHub] flink issue #6242: [FLINK-9711][CLI] Filter only RUNNING jobs when --running...

2018-07-03 Thread satybald
Github user satybald commented on the issue:

https://github.com/apache/flink/pull/6242
  
yep, it's flink 1.5.0. I'm closing PR then.


---


[GitHub] flink issue #6242: [FLINK-9711][CLI] Filter only RUNNING jobs when --running...

2018-07-03 Thread zentol
Github user zentol commented on the issue:

https://github.com/apache/flink/pull/6242
  
see https://issues.apache.org/jira/browse/FLINK-9398


---


[GitHub] flink issue #6242: [FLINK-9711][CLI] Filter only RUNNING jobs when --running...

2018-07-03 Thread zentol
Github user zentol commented on the issue:

https://github.com/apache/flink/pull/6242
  
I take it that happens in 1.5.0? If so, this was already fixed for master 
and 1.5.1.


---


[GitHub] flink issue #6242: [FLINK-9711][CLI] Filter only RUNNING jobs when --running...

2018-07-03 Thread satybald
Github user satybald commented on the issue:

https://github.com/apache/flink/pull/6242
  
I'd agree with you that CREATED, RESTARTING and RUNNING is part of the 
definition of the running job. 

However, CLI does include CANCELED jobs as RUNNING one if --running options 
is provided.

```
flink list --running -m host:8081 
Waiting for response...
-- Running/Restarting Jobs ---
03.07.2018 12:07:26 : b14b50d6b6160035d7c62f135a05b5ea : Enriched 
TrackingClick (CANCELED)
03.07.2018 12:13:39 : 83ab149ad528cfd956da7090543cbc72 : Enriched 
TrackingClick (RUNNING)
--

```



---


[GitHub] flink issue #6242: [FLINK-9711][CLI] Filter only RUNNING jobs when --running...

2018-07-03 Thread zentol
Github user zentol commented on the issue:

https://github.com/apache/flink/pull/6242
  
-1

The PR description is outdated, the CLI only lists jobs in non-terminal 
states as running, i.e. CANCELED, FAILED and FINISHED jobs are not included. 
For all intents and purposes this is the correct behavior, as from a 
user-perspective a job that is currently restarting is effectively still 
running.

Finally, this change would list `SCHEDULED` jobs as terminated, which 
doesn't make any sense.


---