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