sashapolo commented on code in PR #4147:
URL: https://github.com/apache/ignite-3/pull/4147#discussion_r1693145763
##########
modules/cluster-management/src/main/java/org/apache/ignite/internal/cluster/management/ClusterManagementGroupManager.java:
##########
@@ -179,6 +182,31 @@ public ClusterManagementGroupManager(
scheduledExecutor = Executors.newSingleThreadScheduledExecutor(
NamedThreadFactory.create(clusterService.nodeName(),
"cmg-manager", LOG)
);
+
+ var messageCallback = new CmgMessageCallback() {
+ @Override
+ public void onClusterStateMessageReceived(ClusterStateMessage
message, ClusterNode sender, @Nullable Long correlationId) {
+ assert correlationId != null;
Review Comment:
I don't understand how this will help in any way, we have a lot of similar
checks in other places. How are you going to use the sender node?
--
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]
