[GitHub] [solr] dsmiley commented on pull request #1484: SOLR-11685: use INVALID_STATE for ZK state issue in DistributedZkUpdateProcessor

["dsmiley (via GitHub)" <gi...@apache.org>]

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

   On this issue here (but maybe not ShardSplitTest ?), the different error code resulted in the error bubbling all the way back to the client.  My test (not here) was using CloudSolrClient, which retried and it worked.
   
   What you say is interesting though because I'd imagine it'd be useful for the parent shard to retry this, which is the middle-man for our scenarios.  But since the retries there are done there in SolrCmdDistributor specifically, based on your observation, it could never succeed.  At least not for this scenario.  And it didn't retry a 510 code (I saw empirically) anyway.


-- 
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-unsubscribe@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org