You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ma...@apache.org on 2020/09/08 02:36:33 UTC

[lucene-solr] branch reference_impl_dev updated: @798 Bring back test.

This is an automated email from the ASF dual-hosted git repository.

markrmiller pushed a commit to branch reference_impl_dev
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git


The following commit(s) were added to refs/heads/reference_impl_dev by this push:
     new dcfbcbb  @798 Bring back test.
dcfbcbb is described below

commit dcfbcbbc1a123d85cff97e4ca7176244f5b7d7e5
Author: markrmiller@gmail.com <ma...@gmail.com>
AuthorDate: Mon Sep 7 21:36:18 2020 -0500

    @798 Bring back test.
---
 .../core/src/test/org/apache/solr/cloud/DocValuesNotIndexedTest.java | 5 +++--
 solr/test-framework/src/java/org/apache/solr/SolrTestCaseJ4.java     | 2 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/solr/core/src/test/org/apache/solr/cloud/DocValuesNotIndexedTest.java b/solr/core/src/test/org/apache/solr/cloud/DocValuesNotIndexedTest.java
index 9a1f153..1ffa71b 100644
--- a/solr/core/src/test/org/apache/solr/cloud/DocValuesNotIndexedTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/DocValuesNotIndexedTest.java
@@ -65,7 +65,6 @@ import org.slf4j.LoggerFactory;
 
 import com.carrotsearch.randomizedtesting.rules.SystemPropertiesRestoreRule;
 
-@Ignore // nocommit flakey
 public class DocValuesNotIndexedTest extends SolrCloudTestCase {
 
   private static final Logger log = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
@@ -83,9 +82,11 @@ public class DocValuesNotIndexedTest extends SolrCloudTestCase {
 
   @BeforeClass
   public static void createCluster() throws Exception {
+
+    SolrTestCaseJ4.randomizeNumericTypesProperties();
     System.setProperty("managed.schema.mutable", "true");
     configureCluster(2)
-        .addConfig("conf1", TEST_PATH().resolve("configsets").resolve("cloud-managed").resolve("conf"))
+        .addConfig("conf1", configset("cloud-managed"))
         .configure();
 
     // Need enough shards that we have some shards that don't have any docs on them.
diff --git a/solr/test-framework/src/java/org/apache/solr/SolrTestCaseJ4.java b/solr/test-framework/src/java/org/apache/solr/SolrTestCaseJ4.java
index 75053d9..70bbc51 100644
--- a/solr/test-framework/src/java/org/apache/solr/SolrTestCaseJ4.java
+++ b/solr/test-framework/src/java/org/apache/solr/SolrTestCaseJ4.java
@@ -2610,7 +2610,7 @@ public abstract class SolrTestCaseJ4 extends SolrTestCase {
    * @lucene.experimental
    * @lucene.internal
    */
-  private static void randomizeNumericTypesProperties() {
+  public static void randomizeNumericTypesProperties() {
 
     final boolean useDV = random().nextBoolean();
     System.setProperty(NUMERIC_DOCVALUES_SYSPROP, ""+useDV);