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

    https://github.com/apache/flink/pull/6049#discussion_r192699721
  
    --- 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 --
    
    given the existing pattern of separating by `JobStatus` we may want to do 
the same for the remaining jobs.
    
    ```
    
runningJobs.stream().collect(Collectors.groupingBy(JobStatusMessage::getJobState))
                                        .entrySet().stream().sorted(<sort by 
JobStatus>)
                                        
.map(Map.Entry::getValue).flatMap(List::stream).sorted(<sort by 
timestamp>).forEachOrdered(<print>);
    ```


---

Reply via email to