[GitHub] flink pull request #5312: [FLINK-8344][WIP] Add support for HA to RestCluste...

[GJL]

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

    https://github.com/apache/flink/pull/5312#discussion_r162412749
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/highavailability/nonha/standalone/StandaloneHaServices.java
 ---
    @@ -132,4 +137,23 @@ public LeaderElectionService 
getJobManagerLeaderElectionService(JobID jobID) {
                        return new StandaloneLeaderElectionService();
                }
        }
    +
    +   @Override
    +   public LeaderRetrievalService getRestServerLeaderRetriever() {
    --- End diff --
    
    Since `WebMonitorEndpoint` is the instance taking part in leader election 
currently, this should be called `getWebmonitorLeaderRetriever`. 


---