SammyVimes commented on code in PR #769:
URL: https://github.com/apache/ignite-3/pull/769#discussion_r848531096
##########
modules/raft/src/main/java/org/apache/ignite/raft/jraft/JRaftServiceFactory.java:
##########
@@ -31,11 +31,11 @@ public interface JRaftServiceFactory {
/**
* Creates a raft log storage.
*
- * @param uri The log storage uri from {@link NodeOptions#getSnapshotUri()}
+ * @param groupId The name of the raft group.
* @param raftOptions the raft options.
* @return storage to store raft log entires.
*/
- LogStorage createLogStorage(final String uri, final RaftOptions
raftOptions);
+ LogStorage createLogStorage(final String groupId, final RaftOptions
raftOptions);
Review Comment:
All log storages have the same URI in this PR. The only way to distinguish
data in a storage is by raft group id. Hence the change
--
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]
