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: [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]

Reply via email to