epugh commented on code in PR #1951:
URL: https://github.com/apache/solr/pull/1951#discussion_r1335006424
##########
solr/core/src/java/org/apache/solr/cli/RunExampleTool.java:
##########
@@ -869,6 +857,7 @@ protected boolean isValidConfig(File configsetsDir, String
config) {
}
protected Map<String, Object> getNodeStatus(String solrUrl, int maxWaitSecs)
throws Exception {
+ solrUrl += "/solr";
Review Comment:
bummer! But we are in an awkward place where a solrUrl can end with /solr
or with /api ;-) So maybe this pattern becomes more common in varous tools...
Actually, are you sure you need this? Ithought the status tool would deal with
the solrURL.....
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
