You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ct...@apache.org on 2020/02/24 21:55:34 UTC
[accumulo] branch 1.9 updated: Re #1492 Disable unreliable
replication tests (#1524)
This is an automated email from the ASF dual-hosted git repository.
ctubbsii pushed a commit to branch 1.9
in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/1.9 by this push:
new 1775486 Re #1492 Disable unreliable replication tests (#1524)
1775486 is described below
commit 1775486b99c1fdc0b6501973dd82f77d9aaca12d
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Mon Feb 24 16:55:19 2020 -0500
Re #1492 Disable unreliable replication tests (#1524)
Fixes #1226 Disables frequently failing CyclicReplicationIT
---
.../accumulo/test/gc/replication/CloseWriteAheadLogReferencesIT.java | 2 ++
.../org/apache/accumulo/test/replication/CyclicReplicationIT.java | 2 ++
.../org/apache/accumulo/test/replication/FinishedWorkUpdaterIT.java | 2 ++
.../test/replication/GarbageCollectorCommunicatesWithTServersIT.java | 2 ++
.../org/apache/accumulo/test/replication/KerberosReplicationIT.java | 2 ++
.../apache/accumulo/test/replication/MultiInstanceReplicationIT.java | 2 ++
.../apache/accumulo/test/replication/MultiTserverReplicationIT.java | 5 ++---
.../test/replication/RemoveCompleteReplicationRecordsIT.java | 2 ++
.../java/org/apache/accumulo/test/replication/ReplicationIT.java | 2 ++
.../accumulo/test/replication/ReplicationOperationsImplIT.java | 2 ++
.../apache/accumulo/test/replication/ReplicationRandomWalkIT.java | 2 ++
.../apache/accumulo/test/replication/SequentialWorkAssignerIT.java | 2 ++
.../org/apache/accumulo/test/replication/StatusCombinerMacIT.java | 2 ++
.../java/org/apache/accumulo/test/replication/StatusMakerIT.java | 2 ++
.../apache/accumulo/test/replication/UnorderedWorkAssignerIT.java | 2 ++
.../test/replication/UnorderedWorkAssignerReplicationIT.java | 2 ++
.../test/replication/UnusedWalDoesntCloseReplicationStatusIT.java | 2 ++
.../main/java/org/apache/accumulo/test/replication/WorkMakerIT.java | 2 ++
18 files changed, 36 insertions(+), 3 deletions(-)
diff --git a/test/src/main/java/org/apache/accumulo/test/gc/replication/CloseWriteAheadLogReferencesIT.java b/test/src/main/java/org/apache/accumulo/test/gc/replication/CloseWriteAheadLogReferencesIT.java
index 6b2045e..55e9087 100644
--- a/test/src/main/java/org/apache/accumulo/test/gc/replication/CloseWriteAheadLogReferencesIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/gc/replication/CloseWriteAheadLogReferencesIT.java
@@ -57,10 +57,12 @@ import org.apache.hadoop.io.Text;
import org.easymock.EasyMock;
import org.easymock.IAnswer;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
import com.google.common.collect.Iterables;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class CloseWriteAheadLogReferencesIT extends ConfigurableMacBase {
private WrappedCloseWriteAheadLogReferences refs;
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/CyclicReplicationIT.java b/test/src/main/java/org/apache/accumulo/test/replication/CyclicReplicationIT.java
index 29b932e..300d043 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/CyclicReplicationIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/CyclicReplicationIT.java
@@ -58,6 +58,7 @@ import org.apache.accumulo.tserver.replication.AccumuloReplicaSystem;
import org.apache.commons.io.FileUtils;
import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.fs.RawLocalFileSystem;
+import org.junit.Ignore;
import org.junit.Rule;
import org.junit.Test;
import org.junit.experimental.categories.Category;
@@ -68,6 +69,7 @@ import org.slf4j.LoggerFactory;
import com.google.common.collect.Iterables;
+@Ignore("Replication ITs are not stable and not currently maintained")
@Category(MiniClusterOnlyTests.class)
public class CyclicReplicationIT {
private static final Logger log = LoggerFactory.getLogger(CyclicReplicationIT.class);
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/FinishedWorkUpdaterIT.java b/test/src/main/java/org/apache/accumulo/test/replication/FinishedWorkUpdaterIT.java
index f033a7d..d8264f1 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/FinishedWorkUpdaterIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/FinishedWorkUpdaterIT.java
@@ -38,10 +38,12 @@ import org.apache.accumulo.master.replication.FinishedWorkUpdater;
import org.apache.accumulo.server.replication.proto.Replication.Status;
import org.apache.accumulo.test.functional.ConfigurableMacBase;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
import com.google.common.collect.Iterables;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class FinishedWorkUpdaterIT extends ConfigurableMacBase {
private Connector conn;
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/GarbageCollectorCommunicatesWithTServersIT.java b/test/src/main/java/org/apache/accumulo/test/replication/GarbageCollectorCommunicatesWithTServersIT.java
index 6cdab38..5c59b8e 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/GarbageCollectorCommunicatesWithTServersIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/GarbageCollectorCommunicatesWithTServersIT.java
@@ -64,6 +64,7 @@ import org.apache.hadoop.fs.FileSystem;
import org.apache.hadoop.fs.Path;
import org.apache.hadoop.fs.RawLocalFileSystem;
import org.apache.hadoop.io.Text;
+import org.junit.Ignore;
import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -73,6 +74,7 @@ import org.slf4j.LoggerFactory;
* still continue to use it. Checking that no tablet references a WAL is insufficient to determine
* if a WAL will never be used in the future.
*/
+@Ignore("Replication ITs are not stable and not currently maintained")
public class GarbageCollectorCommunicatesWithTServersIT extends ConfigurableMacBase {
private static final Logger log =
LoggerFactory.getLogger(GarbageCollectorCommunicatesWithTServersIT.class);
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/KerberosReplicationIT.java b/test/src/main/java/org/apache/accumulo/test/replication/KerberosReplicationIT.java
index 4a585b6..e4d6027 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/KerberosReplicationIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/KerberosReplicationIT.java
@@ -58,6 +58,7 @@ import org.junit.After;
import org.junit.AfterClass;
import org.junit.Before;
import org.junit.BeforeClass;
+import org.junit.Ignore;
import org.junit.Test;
import org.junit.experimental.categories.Category;
import org.slf4j.Logger;
@@ -68,6 +69,7 @@ import com.google.common.collect.Iterators;
/**
* Ensure that replication occurs using keytabs instead of password (not to mention SASL)
*/
+@Ignore("Replication ITs are not stable and not currently maintained")
@Category(MiniClusterOnlyTests.class)
public class KerberosReplicationIT extends AccumuloITBase {
private static final Logger log = LoggerFactory.getLogger(KerberosIT.class);
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/MultiInstanceReplicationIT.java b/test/src/main/java/org/apache/accumulo/test/replication/MultiInstanceReplicationIT.java
index aea9dd1..9f9974e 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/MultiInstanceReplicationIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/MultiInstanceReplicationIT.java
@@ -68,6 +68,7 @@ import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.fs.RawLocalFileSystem;
import org.junit.After;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -77,6 +78,7 @@ import com.google.common.collect.Iterators;
/**
* Replication tests which start at least two MAC instances and replicate data between them
*/
+@Ignore("Replication ITs are not stable and not currently maintained")
public class MultiInstanceReplicationIT extends ConfigurableMacBase {
private static final Logger log = LoggerFactory.getLogger(MultiInstanceReplicationIT.class);
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/MultiTserverReplicationIT.java b/test/src/main/java/org/apache/accumulo/test/replication/MultiTserverReplicationIT.java
index 5647bbc..5343b32 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/MultiTserverReplicationIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/MultiTserverReplicationIT.java
@@ -36,15 +36,14 @@ import org.apache.accumulo.fate.zookeeper.ZooReader;
import org.apache.accumulo.minicluster.impl.MiniAccumuloConfigImpl;
import org.apache.accumulo.test.functional.ConfigurableMacBase;
import org.apache.hadoop.conf.Configuration;
+import org.junit.Ignore;
import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import com.google.common.collect.Iterables;
-/**
- *
- */
+@Ignore("Replication ITs are not stable and not currently maintained")
public class MultiTserverReplicationIT extends ConfigurableMacBase {
private static final Logger log = LoggerFactory.getLogger(MultiTserverReplicationIT.class);
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/RemoveCompleteReplicationRecordsIT.java b/test/src/main/java/org/apache/accumulo/test/replication/RemoveCompleteReplicationRecordsIT.java
index f3cd45c..163422a 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/RemoveCompleteReplicationRecordsIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/RemoveCompleteReplicationRecordsIT.java
@@ -47,10 +47,12 @@ import org.apache.accumulo.server.replication.proto.Replication.Status;
import org.apache.accumulo.test.functional.ConfigurableMacBase;
import org.easymock.EasyMock;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
import com.google.common.collect.Iterables;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class RemoveCompleteReplicationRecordsIT extends ConfigurableMacBase {
private MockRemoveCompleteReplicationRecords rcrr;
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/ReplicationIT.java b/test/src/main/java/org/apache/accumulo/test/replication/ReplicationIT.java
index 6d435fa..2cc2ccf 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/ReplicationIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/ReplicationIT.java
@@ -100,6 +100,7 @@ import org.apache.hadoop.fs.FileSystem;
import org.apache.hadoop.fs.Path;
import org.apache.hadoop.fs.RawLocalFileSystem;
import org.apache.hadoop.io.Text;
+import org.junit.Ignore;
import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -117,6 +118,7 @@ import com.google.protobuf.TextFormat;
* MockReplicaSystem is used to "fake" the peer instance that we're replicating to. This lets us
* test replication in a functional way without having to worry about two real systems.
*/
+@Ignore("Replication ITs are not stable and not currently maintained")
public class ReplicationIT extends ConfigurableMacBase {
private static final Logger log = LoggerFactory.getLogger(ReplicationIT.class);
private static final long MILLIS_BETWEEN_REPLICATION_TABLE_ONLINE_CHECKS = 5000l;
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/ReplicationOperationsImplIT.java b/test/src/main/java/org/apache/accumulo/test/replication/ReplicationOperationsImplIT.java
index e221495..09ff754 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/ReplicationOperationsImplIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/ReplicationOperationsImplIT.java
@@ -59,10 +59,12 @@ import org.apache.hadoop.io.Text;
import org.apache.thrift.TException;
import org.easymock.EasyMock;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class ReplicationOperationsImplIT extends ConfigurableMacBase {
private static final Logger log = LoggerFactory.getLogger(ReplicationOperationsImplIT.class);
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/ReplicationRandomWalkIT.java b/test/src/main/java/org/apache/accumulo/test/replication/ReplicationRandomWalkIT.java
index 5163edb..ce16978 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/ReplicationRandomWalkIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/ReplicationRandomWalkIT.java
@@ -29,8 +29,10 @@ import org.apache.accumulo.test.functional.ConfigurableMacBase;
import org.apache.accumulo.test.randomwalk.Environment;
import org.apache.accumulo.test.randomwalk.concurrent.Replication;
import org.apache.hadoop.conf.Configuration;
+import org.junit.Ignore;
import org.junit.Test;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class ReplicationRandomWalkIT extends ConfigurableMacBase {
@Override
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/SequentialWorkAssignerIT.java b/test/src/main/java/org/apache/accumulo/test/replication/SequentialWorkAssignerIT.java
index 000302f..2eb6898 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/SequentialWorkAssignerIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/SequentialWorkAssignerIT.java
@@ -43,8 +43,10 @@ import org.apache.accumulo.server.zookeeper.ZooCache;
import org.apache.accumulo.test.functional.ConfigurableMacBase;
import org.apache.hadoop.io.Text;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class SequentialWorkAssignerIT extends ConfigurableMacBase {
private Connector conn;
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/StatusCombinerMacIT.java b/test/src/main/java/org/apache/accumulo/test/replication/StatusCombinerMacIT.java
index cd0dedd..85ff4ce 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/StatusCombinerMacIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/StatusCombinerMacIT.java
@@ -46,10 +46,12 @@ import org.apache.accumulo.server.replication.proto.Replication.Status;
import org.apache.accumulo.server.util.ReplicationTableUtil;
import org.junit.AfterClass;
import org.junit.BeforeClass;
+import org.junit.Ignore;
import org.junit.Test;
import com.google.common.collect.Iterables;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class StatusCombinerMacIT extends SharedMiniClusterBase {
@Override
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/StatusMakerIT.java b/test/src/main/java/org/apache/accumulo/test/replication/StatusMakerIT.java
index b29be49..f93b255 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/StatusMakerIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/StatusMakerIT.java
@@ -56,11 +56,13 @@ import org.apache.hadoop.fs.Path;
import org.apache.hadoop.io.Text;
import org.easymock.EasyMock;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
import com.google.common.collect.Iterables;
import com.google.common.collect.Sets;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class StatusMakerIT extends ConfigurableMacBase {
private Connector conn;
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/UnorderedWorkAssignerIT.java b/test/src/main/java/org/apache/accumulo/test/replication/UnorderedWorkAssignerIT.java
index 58740a4..6508007 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/UnorderedWorkAssignerIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/UnorderedWorkAssignerIT.java
@@ -44,8 +44,10 @@ import org.apache.accumulo.test.functional.ConfigurableMacBase;
import org.apache.hadoop.fs.Path;
import org.apache.hadoop.io.Text;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class UnorderedWorkAssignerIT extends ConfigurableMacBase {
private Connector conn;
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/UnorderedWorkAssignerReplicationIT.java b/test/src/main/java/org/apache/accumulo/test/replication/UnorderedWorkAssignerReplicationIT.java
index 871809c..a034ab4 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/UnorderedWorkAssignerReplicationIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/UnorderedWorkAssignerReplicationIT.java
@@ -68,12 +68,14 @@ import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.fs.RawLocalFileSystem;
import org.junit.After;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import com.google.common.collect.Iterators;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class UnorderedWorkAssignerReplicationIT extends ConfigurableMacBase {
private static final Logger log =
LoggerFactory.getLogger(UnorderedWorkAssignerReplicationIT.class);
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/UnusedWalDoesntCloseReplicationStatusIT.java b/test/src/main/java/org/apache/accumulo/test/replication/UnusedWalDoesntCloseReplicationStatusIT.java
index b849ce2..43b9024 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/UnusedWalDoesntCloseReplicationStatusIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/UnusedWalDoesntCloseReplicationStatusIT.java
@@ -59,10 +59,12 @@ import org.apache.hadoop.fs.FSDataOutputStream;
import org.apache.hadoop.fs.FileSystem;
import org.apache.hadoop.fs.Path;
import org.apache.hadoop.io.Text;
+import org.junit.Ignore;
import org.junit.Test;
import com.google.common.collect.Iterables;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class UnusedWalDoesntCloseReplicationStatusIT extends ConfigurableMacBase {
@Override
diff --git a/test/src/main/java/org/apache/accumulo/test/replication/WorkMakerIT.java b/test/src/main/java/org/apache/accumulo/test/replication/WorkMakerIT.java
index af63368..6aacd14 100644
--- a/test/src/main/java/org/apache/accumulo/test/replication/WorkMakerIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/replication/WorkMakerIT.java
@@ -43,11 +43,13 @@ import org.apache.accumulo.test.functional.ConfigurableMacBase;
import org.apache.hadoop.fs.Path;
import org.apache.hadoop.io.Text;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.Iterables;
+@Ignore("Replication ITs are not stable and not currently maintained")
public class WorkMakerIT extends ConfigurableMacBase {
private Connector conn;