DiscoveryWithServiceDisruptions: some more java docs and todos

tlrx · Oct 2, 2014 · c4866b3 · c4866b3
1 parent 3b38db1
commit c4866b3
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/test/java/org/elasticsearch/discovery/DiscoveryWithServiceDisruptions.java b/src/test/java/org/elasticsearch/discovery/DiscoveryWithServiceDisruptions.java
@@ -410,14 +410,17 @@ public void testIsolateMasterAndVerifyClusterStateConsensus() throws Exception {
     }
 
     /**
-     * Test the we do not loose document whose indexing request was successful, under a randomly selected disruption scheme
+     * Test that we do not loose document whose indexing request was successful, under a randomly selected disruption scheme
      * We also collect & report the type of indexing failures that occur.
+     *
+     * This test is a superset of tests run in the Jepsen test suite, with the exception of versioned updates
      */
     @Test
     // NOTE: if you remove the awaitFix, make sure to port the test to the 1.x branch
     @LuceneTestCase.AwaitsFix(bugUrl = "needs some more work to stabilize")
     @TestLogging("action.index:TRACE,action.get:TRACE,discovery:TRACE,cluster.service:TRACE,indices.recovery:TRACE,indices.cluster:TRACE")
     public void testAckedIndexing() throws Exception {
+        // TODO: add node count randomizaion
         final List<String> nodes = startCluster(3);
 
         assertAcked(prepareCreate("test")
@@ -874,6 +877,7 @@ protected NetworkPartition addRandomIsolation(String isolatedNode) {
     }
 
     private ServiceDisruptionScheme addRandomDisruptionScheme() {
+        // TODO: add partial partitions
         List<ServiceDisruptionScheme> list = Arrays.asList(
                 new NetworkUnresponsivePartition(getRandom()),
                 new NetworkDelaysPartition(getRandom()),