spring-cloud/spring-cloud-netflix

RequestSpecificRetryHandler retries same server for SERVER_THROTTLED

オープン

#2,585 opened on 2017/12/27

 (10 件のコメント) (0 件のリアクション) (0 人の担当者)Java (2,433 件のフォーク)batch import
help wanted

Repository metrics

Stars
 (4,805 個のスター)
PR merge metrics
 (PR metrics pending)

説明

Netflix's RequestSpecificRetryHandler allows to retry on same server for SERVER_THROTTLED error.

When the okToRetryOnAllErrors property is true, the retry handler allows to retry on the same server unconditionally without taking into account the actual error. This is useless when the server responded with SERVER_THROTTLED.

It looks like the if statement at https://github.com/Netflix/ribbon/blob/master/ribbon-core/src/main/java/com/netflix/client/RequestSpecificRetryHandler.java#L58-L65 should be considered first before the other conditions.

I doubt Netflix is willing to change this behavior but we could override it in SpringCloud. What do you think ?

コントリビューターガイド