mlbiscoc commented on code in PR #4219: URL: https://github.com/apache/solr/pull/4219#discussion_r2942911708
########## solr/solrj/src/java/org/apache/solr/client/solrj/impl/CloudSolrClient.java: ########## Review Comment: I only ran into this bug because it was forgotten to pass the the routing info on delete. On the flip side, if this exception didn't occur, deletes would have passed and I never would have known deletes we're being sent in an unoptimized way. One way to catch is to add a log here to a SolrJ client but I think that'd be way to noisy. Maybe a debug log? Or maybe that is the correct way and just never log. -- 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]
