Updates per PR comments

Signed-off-by: Chris Helma <[email protected]>
opensearch-project · Jan 9, 2025 · 7e009a8 · 7e009a8
1 parent 2262d1d
commit 7e009a8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 27 deletions.
diff --git a/...sFromSnapshotMigration/src/test/java/org/opensearch/migrations/bulkload/EndToEndTest.java b/...sFromSnapshotMigration/src/test/java/org/opensearch/migrations/bulkload/EndToEndTest.java
@@ -179,30 +179,5 @@ private void checkDocsWithRouting(
             Assertions.assertEquals("1", routing);
         }
     }
-
-    // private static Stream<Arguments> scenarios_experimental() {
-    //     var scenarios = Stream.<Arguments>builder();
-
-    //     for (var sourceCluster : SupportedClusters.sources()) {
-    //         for (var targetCluster : SupportedClusters.targets_experimental()) {
-    //             scenarios.add(Arguments.of(sourceCluster, targetCluster));
-    //         }
-    //     }
-
-    //     return scenarios.build();
-    // }
-
-    // @ParameterizedTest(name = "Source {0} to Target {1}")
-    // @MethodSource(value = "scenarios_experimental")
-    // public void migrationDocumentsExperimental(
-    //     final SearchClusterContainer.ContainerVersion sourceVersion,
-    //     final SearchClusterContainer.ContainerVersion targetVersion) throws Exception {
-    //     try (
-    //         final var sourceCluster = new SearchClusterContainer(sourceVersion);
-    //         final var targetCluster = new SearchClusterContainer(targetVersion)
-    //     ) {
-    //         migrationDocumentsWithClusters(sourceCluster, targetCluster);
-    //     }
-    // }
-
+
 }
diff --git a/...n/java/org/opensearch/migrations/bulkload/workcoordination/OpenSearchWorkCoordinator.java b/...n/java/org/opensearch/migrations/bulkload/workcoordination/OpenSearchWorkCoordinator.java
@@ -31,7 +31,7 @@
 import org.slf4j.LoggerFactory;
 
 public abstract class OpenSearchWorkCoordinator implements IWorkCoordinator {
-    // Create a stable logger that descendants can use, and we can predictable read from in tests
+    // Create a stable logger that descendants can use, and we can predictably read from in tests
     protected static final Logger log = LoggerFactory.getLogger(OpenSearchWorkCoordinator.class);
 
     public static final String INDEX_NAME = ".migrations_working_state";