dengzhhu653 commented on code in PR #4416:
URL: https://github.com/apache/hive/pull/4416#discussion_r1247315218
##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/leader/StaticLeaderElection.java:
##########
@@ -33,9 +33,9 @@
* If the value is empty, means no election happening and all instances are
leaders,
* otherwise the instance matching the configured hostname will be selected as
a leader.
*/
-public class HostLeaderElection implements LeaderElection<String> {
+public class StaticLeaderElection implements LeaderElection<String> {
Review Comment:
The host leader election is a "static" way to elect a leader, means once if
host is chosen, the leadership cannot switch to other instances even if the
instance is down or the host is crashing, so I would rename it to the
`StaticLeaderElection`, compared to the dynamic leader elelction.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
