[GitHub] [flink] tillrohrmann commented on a change in pull request #9609: [FLINK-13750][client][coordination] Separate HA services between client-side and server-side

[GitBox <gi...@apache.org>]

tillrohrmann commented on a change in pull request #9609: [FLINK-13750][client][coordination] Separate HA services between client-side and server-side
URL: https://github.com/apache/flink/pull/9609#discussion_r320671940
 
 

 ##########
 File path: flink-runtime/src/main/java/org/apache/flink/runtime/highavailability/HighAvailabilityServices.java
 ##########
 @@ -98,6 +98,15 @@
 	 */
 	LeaderRetrievalService getJobManagerLeaderRetriever(JobID jobID, String defaultJobManagerAddress);
 
+	/**
+	 * This retriever should be no longer used in cluster side. The retriever to web monitor
+	 * is only required in client-side and we have a dedicated high-availability services
+	 * for client, named {@link ClientHighAvailabilityServices}. See also FLINK-13750.
 
 Review comment:
   ```suggestion
   	 * for the client, named {@link ClientHighAvailabilityServices}. See also FLINK-13750.
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services