ptupitsyn commented on code in PR #6469:
URL: https://github.com/apache/ignite-3/pull/6469#discussion_r2301573984
##########
modules/client-handler/src/main/java/org/apache/ignite/client/handler/ClientInboundMessageHandler.java:
##########
@@ -301,7 +302,8 @@ public ClientInboundMessageHandler(
BitSet features,
Map<HandshakeExtension, Object> extensions,
Function<String, CompletableFuture<PlatformComputeConnection>>
computeConnectionFunc,
- HandshakeEventLoopSwitcher handshakeEventLoopSwitcher
+ HandshakeEventLoopSwitcher handshakeEventLoopSwitcher,
+ ClusterIdSupplier clusterIdSupplier
Review Comment:
`clusterInfoSupplier` above already has cluster id I think.
##########
modules/api/src/main/java/org/apache/ignite/network/IgniteCluster.java:
##########
@@ -18,13 +18,21 @@
package org.apache.ignite.network;
import java.util.Collection;
+import java.util.UUID;
import java.util.concurrent.CompletableFuture;
import org.jetbrains.annotations.Nullable;
/**
* Represents an Ignite cluster, providing access to the cluster nodes and the
local node.
*/
public interface IgniteCluster {
+ /**
+ * Returns the current cluster ID.
+ *
+ * @return Cluster's current ID.
+ */
+ UUID clusterId();
Review Comment:
```suggestion
UUID id();
```
Thoughts?
--
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: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
