csudharsanan commented on code in PR #2792:
URL: https://github.com/apache/helix/pull/2792#discussion_r1586837657


##########
helix-core/src/main/java/org/apache/helix/HelixAdmin.java:
##########
@@ -415,6 +415,17 @@ void manuallyEnableMaintenanceMode(String clusterName, 
boolean enabled, String r
    */
   ClusterManagementMode getClusterManagementMode(String clusterName);
 
+  /**
+   * Set a list of partitions for an instance to ERROR from any current state.
+   * The partitions could be in any state and setPartitionToError will bring 
them to ERROR
+   * from their current state. A CURRENT_STATE to ERROR state transition is 
required for this.
+   * @param clusterName
+   * @param instanceName
+   * @param resourceName
+   * @param partitionNames
+   */
+  void setPartitionToError(String clusterName, String instanceName, String 
resourceName, List<String> partitionNames);

Review Comment:
   it is public. Updated ZKHelixAdmin to reflect that, and renamed the method 
to setPartitionsToError



-- 
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]

Reply via email to