mkhludnev commented on code in PR #2074:
URL: https://github.com/apache/solr/pull/2074#discussion_r1398277168
##########
solr/core/src/java/org/apache/solr/packagemanager/PackageManager.java:
##########
@@ -1094,9 +1102,8 @@ public Map<String, String> getDeployedCollections(String
packageName) {
new ModifiableSolrParams().add("omitHeader", "true"));
String version = null;
try {
- version =
- JsonPath.parse(paramsJson, PackageUtils.jsonPathConfiguration())
- .read("$['response'].['params'].['PKG_VERSIONS'].['" +
packageName + "'])");
+ String jsonPath = "$['response'].['params'].['PKG_VERSIONS'].['" +
packageName + "'])";
+ version = jsonPathResolve(paramsJson, jsonPath);
} catch (PathNotFoundException ex) {
// Don't worry if PKG_VERSION wasn't found. It just means this
collection was never touched
// by the package manager.
Review Comment:
or it's a redundant bracket in the end of json path
--
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]
