ErickErickson commented on a change in pull request #713: SOLR-12013:
collections API CUSTERSTATUS command fails when configset…
URL: https://github.com/apache/lucene-solr/pull/713#discussion_r292630719
##########
File path: solr/core/src/test/org/apache/solr/cloud/SolrCLIZkUtilsTest.java
##########
@@ -690,28 +690,11 @@ public void testRm() throws Exception {
"-zkHost", zkAddr,
};
- copyConfigUp(configSet, "cloud-subdirs", "rm3");
+ AbstractDistribZkTestBase.copyConfigUp(configSet, "cloud-subdirs", "rm3",
zkAddr);
res =
tool.runTool(SolrCLI.processCommandLineArgs(SolrCLI.joinCommonAndToolOptions(tool.getOptions()),
args));
assertFalse("Should fail when trying to remove /.", res == 0);
}
- // We can use this for testing since the goal is to move "some stuff" up to
ZK.
Review comment:
Refactored this out into a base class
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
