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`
---