You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by gu...@apache.org on 2019/02/19 14:53:43 UTC
[lucene-solr] 06/09: SOLR-13150 fix tests moshe didn't have access
to and remove my print statements from previous commit.
This is an automated email from the ASF dual-hosted git repository.
gus pushed a commit to branch solr-13131
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git
commit 31cacfdd0eb23a2b6109a0cef5e31f122c227c51
Author: Gus Heck <gu...@apache.org>
AuthorDate: Mon Feb 18 22:50:49 2019 -0500
SOLR-13150 fix tests moshe didn't have access to and remove my print statements from previous commit.
---
.../processor/CategoryRoutedAliasUpdateProcessorTest.java | 12 ++----------
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/solr/core/src/test/org/apache/solr/update/processor/CategoryRoutedAliasUpdateProcessorTest.java b/solr/core/src/test/org/apache/solr/update/processor/CategoryRoutedAliasUpdateProcessorTest.java
index 8607a52..f139828 100644
--- a/solr/core/src/test/org/apache/solr/update/processor/CategoryRoutedAliasUpdateProcessorTest.java
+++ b/solr/core/src/test/org/apache/solr/update/processor/CategoryRoutedAliasUpdateProcessorTest.java
@@ -121,19 +121,15 @@ public class CategoryRoutedAliasUpdateProcessorTest extends RoutedAliasUpdatePro
List<String> retrievedConfigSetNames = new ConfigSetAdminRequest.List().process(solrClient).getConfigSets();
List<String> expectedConfigSetNames = Arrays.asList("_default", configName);
- System.out.println("*****************FOO***************");
-
// config sets leak between tests so we can't be any more specific than this on the next 2 asserts
assertTrue("We expect at least 2 configSets",
retrievedConfigSetNames.size() >= expectedConfigSetNames.size());
assertTrue("ConfigNames should include :" + expectedConfigSetNames, retrievedConfigSetNames.containsAll(expectedConfigSetNames));
- System.out.println("*****************FOO2***************");
CollectionAdminRequest.createCategoryRoutedAlias(getAlias(), categoryField,
CollectionAdminRequest.createCollection("_unused_", configName, 1, 1)
- .setMaxShardsPerNode(2))
+ .setMaxShardsPerNode(2)).setMaxCardinality(20)
.process(solrClient);
- System.out.println("*****************FOO3***************");
addDocsAndCommit(true,
newDoc(somethingInChinese),
newDoc(somethingInHebrew),
@@ -142,13 +138,9 @@ public class CategoryRoutedAliasUpdateProcessorTest extends RoutedAliasUpdatePro
newDoc(somethingInGreek),
newDoc(somethingInGujarati));
- System.out.println("*****************FOO4***************");
-
// Note Gujarati not listed, because it duplicates hebrew.
assertInvariants(collectionChinese, collectionHebrew, collectionThai, collectionArabic, collectionGreek);
- System.out.println("*****************FOO5***************");
-
assertColHasDocCount(collectionChinese, 1);
assertColHasDocCount(collectionHebrew, 2);
assertColHasDocCount(collectionThai, 1);
@@ -305,7 +297,7 @@ public class CategoryRoutedAliasUpdateProcessorTest extends RoutedAliasUpdatePro
final int numReplicas = 1 + random().nextInt(3);
CollectionAdminRequest.createCategoryRoutedAlias(getAlias(), categoryField,
CollectionAdminRequest.createCollection("_unused_", configName, numShards, numReplicas)
- .setMaxShardsPerNode(numReplicas))
+ .setMaxShardsPerNode(numReplicas)).setMaxCardinality(20)
.process(solrClient);
// cause some collections to be created