shahryart opened a new pull request, #3238: URL: https://github.com/apache/celeborn/pull/3238
### What changes were proposed in this pull request? Add retries to registerShuffle for lifecycleManager to address failures from FallbackPolicyRunner and/or other manager issues. While we do retry connection failures to manager, this addresses failures such as no active manager for a short period of time as well. We add three separate configs: 1. retryEnabled 2. maxRetries 3. retryWaitTime ### Why are the changes needed? We were running into issues in production with manager not being active for a short period of time, which results in application failures. Adding retries fixes these failures. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Existing unit tests. Running jobs in our pre-prod environment. -- 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]
