gerlowskija commented on code in PR #1286:
URL: https://github.com/apache/solr/pull/1286#discussion_r1087098331
##########
solr/core/src/java/org/apache/solr/core/SolrCore.java:
##########
@@ -225,7 +226,7 @@ public class SolrCore implements SolrInfoBean, Closeable {
private final Date startTime = new Date();
private final long startNanoTime = System.nanoTime();
private final RequestHandlers reqHandlers;
- private final ApplicationHandler jerseyAppHandler;
+ private final RefCounted<ApplicationHandler> appHandlerForConfigSet;
Review Comment:
I removed 'jersey' from the name out of an awareness that we might swap
Jersey out for some other JAX-RS implementation down the road, and the less we
reference the name of the specific implementation the better.
But reading this again now I think I'm ready to flip-flop back to
'jerseyAppHandler'. Idk what I was thinking with the 'ForConfigSet' bit. Done
--
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
