dsmiley commented on PR #3418:
URL: https://github.com/apache/solr/pull/3418#issuecomment-3621680527

   Please take a look Sonu.  I also refactored SearchHandler a bit to have a 
clearer flow / readability and also extensibility.  I'd like to merge the 
refactoring portions of this PR in one go, then the rest of this here.  Like... 
I could extract all that to be under SOLR-17982.
   
   A TODO I didn't get to tonight is testing the different forceDistrib API 
with a custom SearchHandler subclass.  Both testForceDistrib methods are there 
but need to be modified.  I suspect shards.info may reflect what we want to see.


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