gerlowskija commented on code in PR #2681:
URL: https://github.com/apache/solr/pull/2681#discussion_r1737662158


##########
solr/server/solr/configsets/_default/conf/solrconfig.xml:
##########
@@ -656,6 +656,10 @@
 
   <!-- Primary search handler, expected by most clients, examples and UI 
frameworks -->
   <requestHandler name="/select" class="solr.SearchHandler">
+    <shardHandlerFactory name="shardHandlerFactory" 
class="${shardHandlerFactory:ParallelHttpShardHandlerFactory}">

Review Comment:
   Yep - kindof handy for overriding the solr.xml definition on a 
per-collection basis.  The docs on ShardHandler overall could really use some 
beefing up - gonna try to squeeze that into this PR if I can.
   
   Anyways, I agree - no need to have this in our default configset (even as a 
commented out example).  Will move it over to solr.xml.



-- 
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

Reply via email to