gerlowskija commented on code in PR #975:
URL: https://github.com/apache/solr/pull/975#discussion_r959835650
##########
solr/core/src/java/org/apache/solr/api/V2HttpCall.java:
##########
@@ -397,7 +399,7 @@ protected void handleAdmin(SolrQueryResponse solrResp) {
@Override
protected void executeCoreRequest(SolrQueryResponse rsp) {
if (api == null) {
- invokeJerseyRequest(cores, core, core.getApplicationHandler());
+ invokeJerseyRequest(cores, core, core.getApplicationHandler(), rsp);
Review Comment:
I found myself confusing the names every time I came into this class. I
held off on renaming variables too much in fear of muddying the diff, but I'm
happy to do it here if you think it'd help more than it hurts the review here?
--
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: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
