dsmiley commented on code in PR #3238: URL: https://github.com/apache/solr/pull/3238#discussion_r1997661934
########## solr/solrj/src/java/org/apache/solr/client/solrj/SolrRequest.java: ########## @@ -210,6 +236,16 @@ public boolean requiresCollection() { return false; } + /** + * Indicates if clients should make attempts to route this request to a shard leader, overriding + * typical client routing preferences for requests. Defaults to true. + * + * @see CloudSolrClient#isUpdatesToLeaders + */ + public boolean shouldSendToLeaders() { Review Comment: I looked at this closer just now. `shards.preference` routes the entire request whereas `IsUpdateRequest.isSendToLeaders` is about each document in the request, individually routing it. So not the same. Any way, I don't think it makes sense for this to be on SolrRequest generally. -- 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