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