sashapolo commented on code in PR #7937:
URL: https://github.com/apache/ignite-3/pull/7937#discussion_r3043281531
##########
modules/raft/src/test/java/org/apache/ignite/raft/jraft/storage/logit/HybridLogStorageTest.java:
##########
@@ -35,53 +43,97 @@
import org.apache.ignite.raft.jraft.option.RaftOptions;
import org.apache.ignite.raft.jraft.storage.BaseStorageTest;
import org.apache.ignite.raft.jraft.storage.LogStorage;
-import org.apache.ignite.raft.jraft.storage.impl.LocalLogStorage;
import org.apache.ignite.raft.jraft.storage.logit.option.StoreOptions;
import org.apache.ignite.raft.jraft.storage.logit.storage.HybridLogStorage;
import org.apache.ignite.raft.jraft.test.TestUtils;
+import org.jetbrains.annotations.Nullable;
import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+@ExtendWith(ConfigurationExtension.class)
class HybridLogStorageTest extends BaseStorageTest {
private static final String STORAGE_RELATIVE_PATH = "log";
private static final String NEW_STORAGE_RELATIVE_PATH = "new";
+ private static final String GROUP_ID = "1_part_1";
+
@Test
public void testTransferLogStorage() {
+ Path storagePath = getStoragePath();
+
+ LogStorageManager oldStorageFactory = new
LogitLogStorageManager("test", storeOptions(), storagePath.resolve("old"));
Review Comment:
Please don't use LogitLogStorageManager, it's going to be removed in the
future
--
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]
