SOLR-12028: BadApple and AwaitsFix annotations usage

This commit is contained in:
Erick Erickson 2018-04-12 17:58:32 -07:00
parent 2a2a0b6acd
commit 0014f3af88
16 changed files with 21 additions and 2 deletions

View File

@ -2221,6 +2221,7 @@ public class TestIndexSorting extends LuceneTestCase {
} }
// pits index time sorting against query time sorting // pits index time sorting against query time sorting
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testRandom3() throws Exception { public void testRandom3() throws Exception {
int numDocs = atLeast(1000); int numDocs = atLeast(1000);
List<RandomDoc> docs = new ArrayList<>(); List<RandomDoc> docs = new ArrayList<>();

View File

@ -108,6 +108,7 @@ public class TestDistributedSearch extends BaseDistributedSearchTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void test() throws Exception { public void test() throws Exception {
QueryResponse rsp = null; QueryResponse rsp = null;
int backupStress = stress; // make a copy so we can restore int backupStress = stress; // make a copy so we can restore

View File

@ -46,6 +46,7 @@ public class AddReplicaTest extends SolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void test() throws Exception { public void test() throws Exception {
cluster.waitForAllNodes(5000); cluster.waitForAllNodes(5000);
String collection = "addreplicatest_coll"; String collection = "addreplicatest_coll";

View File

@ -240,6 +240,7 @@ public class AliasIntegrationTest extends SolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testModifyPropertiesV1() throws Exception { public void testModifyPropertiesV1() throws Exception {
// note we don't use TZ in this test, thus it's UTC // note we don't use TZ in this test, thus it's UTC
final String aliasName = getTestName(); final String aliasName = getTestName();

View File

@ -112,6 +112,7 @@ public class ChaosMonkeyNothingIsSafeTest extends AbstractFullDistribZkTestBase
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void test() throws Exception { public void test() throws Exception {
// None of the operations used here are particularly costly, so this should work. // None of the operations used here are particularly costly, so this should work.
// Using this low timeout will also help us catch index stalling. // Using this low timeout will also help us catch index stalling.

View File

@ -99,6 +99,7 @@ public class CreateRoutedAliasTest extends SolrCloudTestCase {
// This is a fairly complete test where we set many options and see that it both affected the created // This is a fairly complete test where we set many options and see that it both affected the created
// collection and that the alias metadata was saved accordingly // collection and that the alias metadata was saved accordingly
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testV2() throws Exception { public void testV2() throws Exception {
// note we don't use TZ in this test, thus it's UTC // note we don't use TZ in this test, thus it's UTC
final String aliasName = getTestName(); final String aliasName = getTestName();
@ -180,6 +181,7 @@ public class CreateRoutedAliasTest extends SolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testV1() throws Exception { public void testV1() throws Exception {
final String aliasName = getTestName(); final String aliasName = getTestName();
final String baseUrl = cluster.getRandomJetty(random()).getBaseUrl().toString(); final String baseUrl = cluster.getRandomJetty(random()).getBaseUrl().toString();
@ -223,6 +225,7 @@ public class CreateRoutedAliasTest extends SolrCloudTestCase {
// TZ should not affect the first collection name if absolute date given for start // TZ should not affect the first collection name if absolute date given for start
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testTimezoneAbsoluteDate() throws Exception { public void testTimezoneAbsoluteDate() throws Exception {
final String aliasName = getTestName(); final String aliasName = getTestName();
try (SolrClient client = getCloudSolrClient(cluster)) { try (SolrClient client = getCloudSolrClient(cluster)) {
@ -241,6 +244,7 @@ public class CreateRoutedAliasTest extends SolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testCollectionNamesMustBeAbsent() throws Exception { public void testCollectionNamesMustBeAbsent() throws Exception {
CollectionAdminRequest.createCollection("collection1meta", "_default", 2, 1).process(cluster.getSolrClient()); CollectionAdminRequest.createCollection("collection1meta", "_default", 2, 1).process(cluster.getSolrClient());
CollectionAdminRequest.createCollection("collection2meta", "_default", 1, 1).process(cluster.getSolrClient()); CollectionAdminRequest.createCollection("collection2meta", "_default", 1, 1).process(cluster.getSolrClient());

View File

@ -290,6 +290,7 @@ public class DeleteReplicaTest extends SolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void deleteReplicaOnIndexing() throws Exception { public void deleteReplicaOnIndexing() throws Exception {
final String collectionName = "deleteReplicaOnIndexing"; final String collectionName = "deleteReplicaOnIndexing";
CollectionAdminRequest.createCollection(collectionName, "conf", 1, 2) CollectionAdminRequest.createCollection(collectionName, "conf", 1, 2)

View File

@ -77,6 +77,7 @@ public class TestCloudRecovery extends SolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void leaderRecoverFromLogOnStartupTest() throws Exception { public void leaderRecoverFromLogOnStartupTest() throws Exception {
AtomicInteger countReplayLog = new AtomicInteger(0); AtomicInteger countReplayLog = new AtomicInteger(0);
DirectUpdateHandler2.commitOnClose = false; DirectUpdateHandler2.commitOnClose = false;

View File

@ -30,6 +30,7 @@ import org.apache.solr.util.MockCoreContainer.MockCoreDescriptor;
import org.apache.solr.util.TimeOut; import org.apache.solr.util.TimeOut;
import org.apache.zookeeper.KeeperException; import org.apache.zookeeper.KeeperException;
import org.apache.zookeeper.data.Stat; import org.apache.zookeeper.data.Stat;
import org.junit.Test;
/** /**
* Test for {@link LeaderInitiatedRecoveryThread} * Test for {@link LeaderInitiatedRecoveryThread}
@ -43,6 +44,8 @@ public class TestLeaderInitiatedRecoveryThread extends AbstractFullDistribZkTest
fixShardCount(2); fixShardCount(2);
} }
@Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testPublishDownState() throws Exception { public void testPublishDownState() throws Exception {
waitForRecoveriesToFinish(true); waitForRecoveriesToFinish(true);

View File

@ -93,6 +93,7 @@ public class TestStressInPlaceUpdates extends AbstractFullDistribZkTestBase {
@Test @Test
@ShardsFixed(num = 3) @ShardsFixed(num = 3)
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void stressTest() throws Exception { public void stressTest() throws Exception {
waitForRecoveriesToFinish(true); waitForRecoveriesToFinish(true);

View File

@ -172,6 +172,7 @@ public class TestCollectionsAPIViaSolrCloudCluster extends SolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testCollectionCreateWithoutCoresThenDelete() throws Exception { public void testCollectionCreateWithoutCoresThenDelete() throws Exception {
final String collectionName = "testSolrCloudCollectionWithoutCores"; final String collectionName = "testSolrCloudCollectionWithoutCores";

View File

@ -414,6 +414,7 @@ public class ComputePlanActionTest extends SolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testSelectedCollections() throws Exception { public void testSelectedCollections() throws Exception {
log.info("Found number of jetties: {}", cluster.getJettySolrRunners().size()); log.info("Found number of jetties: {}", cluster.getJettySolrRunners().size());
AssertingTriggerAction.expectedNode = null; AssertingTriggerAction.expectedNode = null;

View File

@ -56,6 +56,7 @@ public class ScheduledTriggerTest extends SolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testTrigger() throws Exception { public void testTrigger() throws Exception {
CoreContainer container = cluster.getJettySolrRunners().get(0).getCoreContainer(); CoreContainer container = cluster.getJettySolrRunners().get(0).getCoreContainer();

View File

@ -327,6 +327,7 @@ public class TestTriggerIntegration extends SimSolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testNodeAddedTriggerRestoreState() throws Exception { public void testNodeAddedTriggerRestoreState() throws Exception {
// for this test we want to update the trigger so we must assert that the actions were created twice // for this test we want to update the trigger so we must assert that the actions were created twice
TestTriggerIntegration.actionInitCalled = new CountDownLatch(2); TestTriggerIntegration.actionInitCalled = new CountDownLatch(2);

View File

@ -242,8 +242,7 @@ public class AutoscalingHistoryHandlerTest extends SolrCloudTestCase {
} }
@Test @Test
// commented out 26-Mar-2018 @BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
//@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028")
public void testHistory() throws Exception { public void testHistory() throws Exception {
waitForState("Timed out wait for collection be active", COLL_NAME, waitForState("Timed out wait for collection be active", COLL_NAME,
clusterShape(1, 3)); clusterShape(1, 3));

View File

@ -267,6 +267,7 @@ public class TestCollectionStateWatchers extends SolrCloudTestCase {
} }
@Test @Test
@BadApple(bugUrl="https://issues.apache.org/jira/browse/SOLR-12028") // 09-Apr-2018
public void testDeletionsTriggerWatches() throws Exception { public void testDeletionsTriggerWatches() throws Exception {
CollectionAdminRequest.createCollection("tobedeleted", "config", 1, 1) CollectionAdminRequest.createCollection("tobedeleted", "config", 1, 1)
.process(cluster.getSolrClient()); .process(cluster.getSolrClient());