Github user tzulitai commented on a diff in the pull request:
https://github.com/apache/flink/pull/4688#discussion_r140193760
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/legacy/CurrentJobsOverviewHandler.java
---
@@ -69,6 +73,13 @@ public CurrentJobsOverviewHandler(
}
@Override
+ public CompletableFuture<MultipleJobsDetails>
handleRequest(HandlerRequest<EmptyRequestBody, EmptyMessageParameters> request,
DispatcherGateway gateway) {
+ CompletableFuture<MultipleJobsDetails> jobDetailsFuture =
gateway.requestJobDetails(timeout);
+
+ return jobDetailsFuture;
--- End diff --
nit: might as well just make this a one-liner:
`return gateway.requestJobDetails(timeout);`
---
