epugh commented on code in PR #1182:
URL: https://github.com/apache/solr/pull/1182#discussion_r1166745952
##
solr/core/src/java/org/apache/solr/util/SolrCLI.java:
##
@@ -1271,14 +1271,13 @@ public static boolean safeCheckCoreExists(String
solrUrl, String coreName) {
epugh commented on code in PR #1182:
URL: https://github.com/apache/solr/pull/1182#discussion_r1162638083
##
solr/core/src/test/org/apache/solr/util/ApiToolTest.java:
##
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor l
epugh commented on code in PR #1182:
URL: https://github.com/apache/solr/pull/1182#discussion_r1158360488
##
solr/core/src/java/org/apache/solr/packagemanager/DefaultPackageRepository.java:
##
@@ -101,16 +106,21 @@ public Path download(String artifactName) throws
SolrException,
epugh commented on code in PR #1182:
URL: https://github.com/apache/solr/pull/1182#discussion_r1158359743
##
solr/core/src/java/org/apache/solr/packagemanager/DefaultPackageRepository.java:
##
@@ -101,16 +106,21 @@ public Path download(String artifactName) throws
SolrException,
epugh commented on code in PR #1182:
URL: https://github.com/apache/solr/pull/1182#discussion_r1127911487
##
solr/core/src/java/org/apache/solr/packagemanager/PackageManager.java:
##
@@ -230,15 +229,12 @@ public List
fetchInstalledPackageInstances() throws SolrExc
public Map
epugh commented on code in PR #1182:
URL: https://github.com/apache/solr/pull/1182#discussion_r1059415280
##
solr/core/src/java/org/apache/solr/packagemanager/PackageManager.java:
##
@@ -228,20 +230,20 @@ public List
fetchInstalledPackageInstances() throws SolrExc
public Map
epugh commented on code in PR #1182:
URL: https://github.com/apache/solr/pull/1182#discussion_r1025532208
##
solr/core/src/java/org/apache/solr/util/PackageTool.java:
##
@@ -360,15 +366,19 @@ private String getZkHost(CommandLine cli) throws
Exception {
String zkHost = cli.
