junkaixue commented on a change in pull request #1807:
URL: https://github.com/apache/helix/pull/1807#discussion_r660194035
##########
File path: helix-core/src/main/java/org/apache/helix/HelixAdmin.java
##########
@@ -381,6 +381,17 @@ void manuallyEnableMaintenanceMode(String clusterName,
boolean enabled, String r
*/
void setClusterManagementMode(ClusterManagementModeRequest request);
+ /**
+ * Gets cluster management status {@link ClusterManagementMode}: what mode
the cluster is and
+ * whether the cluster has fully reached to that mode.
+ *
+ * @param clusterName cluster name
+ * @return {@link ClusterManagementMode}
+ */
+ default ClusterManagementMode getClusterManagementMode(String clusterName) {
Review comment:
HelixAdmin is something not designed to let user implement. Even if they
have different plugin, they should extend our implemented class.
--
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]
