You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2022/11/22 08:16:38 UTC

[GitHub] [druid] gianm opened a new pull request, #13413: ServiceClient: More robust redirect handling.

gianm opened a new pull request, #13413:
URL: https://github.com/apache/druid/pull/13413

   Detects self-redirects, redirect loops, long redirect chains, and redirects to unknown servers. Treat all of these cases as an unavailable service, retrying if the retry policy allows it.
   
   Previously, some of these cases would lead to a prompt, unretryable error. This caused clients contacting an Overlord during a leader change to fail with error messages like:
   
   ```
     org.apache.druid.rpc.RpcException: Service [overlord] redirected too many times
   ```
   
   Additionally, a slight refactor of callbacks in ServiceClientImpl improves readability of the flow through onSuccess.


-- 
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: commits-unsubscribe@druid.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] kfaraz merged pull request #13413: ServiceClient: More robust redirect handling.

Posted by GitBox <gi...@apache.org>.
kfaraz merged PR #13413:
URL: https://github.com/apache/druid/pull/13413


-- 
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: commits-unsubscribe@druid.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org