From ed99da850e72c316fb930d714683e50db536d62c Mon Sep 17 00:00:00 2001 From: Bharat Viswanadham Date: Sun, 10 Feb 2019 19:17:08 -0800 Subject: [PATCH] HDDS-1078. TestRatisPipelineProvider failing because of node count mismatch. Contributed by Mukul Kumar Singh. --- .../hdds/scm/pipeline/TestRatisPipelineProvider.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/hdds/scm/pipeline/TestRatisPipelineProvider.java b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/hdds/scm/pipeline/TestRatisPipelineProvider.java index 4a59831701..6f4934f32a 100644 --- a/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/hdds/scm/pipeline/TestRatisPipelineProvider.java +++ b/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/hdds/scm/pipeline/TestRatisPipelineProvider.java @@ -64,10 +64,11 @@ public void testCreatePipelineWithFactor() throws IOException { factor = HddsProtos.ReplicationFactor.ONE; Pipeline pipeline1 = provider.create(factor); stateManager.addPipeline(pipeline1); - // New pipeline should not overlap with the previous created pipeline - Assert.assertTrue( - CollectionUtils.intersection(pipeline.getNodes(), pipeline1.getNodes()) - .isEmpty()); + // New pipeline should overlap with the previous created pipeline, + // and one datanode should overlap between the two types. + Assert.assertEquals( + CollectionUtils.intersection(pipeline.getNodes(), + pipeline1.getNodes()).size(), 1); Assert.assertEquals(pipeline1.getType(), HddsProtos.ReplicationType.RATIS); Assert.assertEquals(pipeline1.getFactor(), factor); Assert.assertEquals(pipeline1.getPipelineState(),