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