YARN-4113. RM should respect retry-interval when uses RetryPolicies.RETRY_FOREVER. (Sunil G via wangda)
(cherry picked from commit b00392dd9c
)
This commit is contained in:
parent
52f343598e
commit
fafcb4ac4e
|
@ -813,6 +813,9 @@ Release 2.8.0 - UNRELEASED
|
||||||
|
|
||||||
YARN-4167. NPE on RMActiveServices#serviceStop when store is null. (Bibin A Chundatt via rohithsharmaks)
|
YARN-4167. NPE on RMActiveServices#serviceStop when store is null. (Bibin A Chundatt via rohithsharmaks)
|
||||||
|
|
||||||
|
YARN-4113. RM should respect retry-interval when uses RetryPolicies.RETRY_FOREVER.
|
||||||
|
(Sunil G via wangda)
|
||||||
|
|
||||||
Release 2.7.2 - UNRELEASED
|
Release 2.7.2 - UNRELEASED
|
||||||
|
|
||||||
INCOMPATIBLE CHANGES
|
INCOMPATIBLE CHANGES
|
||||||
|
|
|
@ -232,7 +232,8 @@ public class RMProxy<T> {
|
||||||
|
|
||||||
RetryPolicy retryPolicy = null;
|
RetryPolicy retryPolicy = null;
|
||||||
if (waitForEver) {
|
if (waitForEver) {
|
||||||
retryPolicy = RetryPolicies.RETRY_FOREVER;
|
retryPolicy = RetryPolicies.retryForeverWithFixedSleep(
|
||||||
|
rmConnectionRetryIntervalMS, TimeUnit.MILLISECONDS);
|
||||||
} else {
|
} else {
|
||||||
retryPolicy =
|
retryPolicy =
|
||||||
RetryPolicies.retryUpToMaximumTimeWithFixedSleep(rmConnectWaitMS,
|
RetryPolicies.retryUpToMaximumTimeWithFixedSleep(rmConnectWaitMS,
|
||||||
|
|
|
@ -63,7 +63,8 @@ public class ServerProxy {
|
||||||
RetryPolicy retryPolicy = null;
|
RetryPolicy retryPolicy = null;
|
||||||
if (maxWaitTime == -1) {
|
if (maxWaitTime == -1) {
|
||||||
// wait forever.
|
// wait forever.
|
||||||
retryPolicy = RetryPolicies.RETRY_FOREVER;
|
retryPolicy = RetryPolicies.retryForeverWithFixedSleep(retryIntervalMS,
|
||||||
|
TimeUnit.MILLISECONDS);
|
||||||
} else {
|
} else {
|
||||||
retryPolicy =
|
retryPolicy =
|
||||||
RetryPolicies.retryUpToMaximumTimeWithFixedSleep(maxWaitTime,
|
RetryPolicies.retryUpToMaximumTimeWithFixedSleep(maxWaitTime,
|
||||||
|
|
Loading…
Reference in New Issue