Github user zentol commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6049#discussion_r192837393
  
    --- Diff: 
flink-clients/src/main/java/org/apache/flink/client/cli/CliFrontend.java ---
    @@ -474,6 +482,19 @@ protected void list(String[] args) throws Exception {
                                
System.out.println("--------------------------------------------------------------");
                        }
                }
    +           if (showRemaining) {
    +                   if (remainingJobs.size() != 0) {
    +                           remainingJobs.sort(startTimeComparator);
    --- End diff --
    
    only for the last case, specifically this would separate `FAILED`, 
`COMPLETED` and `CANCELED` jobs.
    
    That said the other sections aren't that fine-grained either. That reminds 
me, we could name the last section `terminated` instead of `remaining`.


---

Reply via email to