Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/5430#discussion_r168165205
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/webmonitor/WebMonitorEndpoint.java
---
@@ -593,6 +617,8 @@ public void shutdown(Time timeout) {
} catch (Exception e) {
log.warn("Error while stopping leaderElectionService",
e);
}
+
+ super.shutdown(timeout);
--- End diff --
Why is it better to move it to the end?---
