From c4866b3f03111eac48cbedef5f955c9fde94d4d0 Mon Sep 17 00:00:00 2001 From: Boaz Leskes Date: Thu, 2 Oct 2014 13:29:44 +0200 Subject: [PATCH] DiscoveryWithServiceDisruptions: some more java docs and todos --- .../discovery/DiscoveryWithServiceDisruptions.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/test/java/org/elasticsearch/discovery/DiscoveryWithServiceDisruptions.java b/src/test/java/org/elasticsearch/discovery/DiscoveryWithServiceDisruptions.java index 9095f5ec785..5576fca59e2 100644 --- a/src/test/java/org/elasticsearch/discovery/DiscoveryWithServiceDisruptions.java +++ b/src/test/java/org/elasticsearch/discovery/DiscoveryWithServiceDisruptions.java @@ -410,14 +410,17 @@ public class DiscoveryWithServiceDisruptions extends ElasticsearchIntegrationTes } /** - * 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 nodes = startCluster(3); assertAcked(prepareCreate("test") @@ -874,6 +877,7 @@ public class DiscoveryWithServiceDisruptions extends ElasticsearchIntegrationTes } private ServiceDisruptionScheme addRandomDisruptionScheme() { + // TODO: add partial partitions List list = Arrays.asList( new NetworkUnresponsivePartition(getRandom()), new NetworkDelaysPartition(getRandom()),