csudharsanan commented on code in PR #3043:
URL: https://github.com/apache/helix/pull/3043#discussion_r2110056771
##########
helix-core/src/main/java/org/apache/helix/util/MessageUtil.java:
##########
@@ -136,4 +136,32 @@ private static Message
createStateTransitionMessage(Message.MessageType messageT
return message;
}
+
+ /**
+ * Create a state transition message with currentReplicaNumber for
prioritization
+ * @param msgSender message sender
+ * @param sessionId session id
+ * @param resource resource
+ * @param partitionName partition name
+ * @param instanceName instance name
+ * @param fromState from state
+ * @param toState to state
+ * @param sessionIdForInstance session id for instance
+ * @param stateModelDefName state model def name
+ * @param currentReplicaNumber the current replica number (for
prioritization)
+ * @return message
+ */
+ public static Message createStateTransitionMessage(String msgSender, String
sessionId,
Review Comment:
thanks for pointing this out. I've reformatted the entire class to make it
backwards compatible and renamed the private methods for readability/.
--
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]
