sanpwc commented on a change in pull request #400:
URL: https://github.com/apache/ignite-3/pull/400#discussion_r738132043
##########
File path:
modules/raft-client/src/main/java/org/apache/ignite/raft/client/service/RaftGroupService.java
##########
@@ -226,4 +227,9 @@
* Shutdown and cleanup resources for this instance.
*/
void shutdown();
+
+ /**
+ * @return Cluster service.
+ */
+ ClusterService clusterService();
Review comment:
It seems confusing to have ClusterService provider within
RaftGroupService. Seems that it's only used in tests where you have full
control about cluster services that are used to instantiate
raftGroupServiceImpls?
--
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]
