Gerrrr commented on PR #17768: URL: https://github.com/apache/iceberg/pull/17768#issuecomment-6756138494
Hey @uros-b @waterWang! We're hitting #17756 in production as well. Since this PR has been inactive for a while, I'd like to pick it up as suggested above, with co-author credit to @waterWang. Happy to hand it back if you'd rather continue. My understanding is that the following things need to be changed for this PR to get merged: 1. Fix the spotless check. 2. Replace `core:` with `Core:` in the PR title. 3. Only retry on an actual retry failure. 4. Use exponential backoff from 1s to 60s instrad of a fixed 60s. Does this match your expectations? Question: Should permanent failures like 401/400 from the token endpoint stop the retries? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
