iamsanjay commented on code in PR #5098:
URL: https://github.com/apache/solr/pull/5098#discussion_r4489773547


##########
solr/core/src/test/org/apache/solr/cloud/RecoveryAfterSoftCommitTest.java:
##########
@@ -98,12 +99,21 @@ public void test() throws Exception {
     proxy.close();
 
     // add more than ULOG_NUM_RECORDS_TO_KEEP docs so that peer sync cannot be 
used for recovery
+    // Use a leaders-only client for these adds: closing the proxy also cuts 
the client's pooled

Review Comment:
   @nick-boss-tech actually this may not be leaders-only but leader first, 
there is another flag that need to be set which make sure the request will only 
go to leader. 



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