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

    https://github.com/apache/flink/pull/6049#discussion_r192696972
  
    --- Diff: 
flink-clients/src/main/java/org/apache/flink/client/cli/CliFrontend.java ---
    @@ -436,15 +440,19 @@ protected void list(String[] args) throws Exception {
     
                final List<JobStatusMessage> runningJobs = new ArrayList<>();
                final List<JobStatusMessage> scheduledJobs = new ArrayList<>();
    +           final List<JobStatusMessage> remainingJobs = new ArrayList<>();
                jobDetails.forEach(details -> {
                        if (details.getJobState() == JobStatus.CREATED) {
                                scheduledJobs.add(details);
    -                   } else {
    +                   } else if (details.getJobState() == JobStatus.RUNNING
    +                           || details.getJobState() == 
JobStatus.RESTARTING){
                                runningJobs.add(details);
    +                   } else {
    +                           remainingJobs.add(details);
                        }
                });
     
    -           if (running) {
    +           if (showRunning) {
    --- End diff --
    
    after the change in `ListOptions`: `showRunning || showAll`


---

Reply via email to