nizhikov commented on a change in pull request #7358: IGNITE-12614 : Disallow silent deactivation of cluster to prevent in-mem data loss. URL: https://github.com/apache/ignite/pull/7358#discussion_r376387283
########## File path: modules/core/src/main/java/org/apache/ignite/mxbean/IgniteMXBean.java ########## @@ -380,23 +376,30 @@ */ @MXBeanDescription("Pings node with given node ID to see whether it is alive. " + "Returns whether or not node is alive.") - @MXBeanParametersNames( - "nodeId" - ) - @MXBeanParametersDescriptions( - "String presentation of node ID. See java.util.UUID class for details." - ) + @MXBeanParametersNames("nodeId") + @MXBeanParametersDescriptions("String presentation of node ID. See java.util.UUID class for details.") public boolean pingNode(String nodeId); + /** Activates cluster. */ + @MXBeanDescription("Execute activation process.") + @MXBeanParametersNames("active") + public void activate(); + + /** Deactivates cluster. */ + @MXBeanDescription("Execute deactivation process.") + @MXBeanParametersNames("force") Review comment: JMX bean is a public API, so we should preserve backward compatibility. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services