8000 Further revise retry logic to avoid stacked retry overruns by cmgrote · Pull Request #2177 · atlanhq/atlan-java · GitHub
[go: up one dir, main page]

Skip to content

Further revise retry logic to avoid stacked retry overruns#2177

Merged
cmgrote merged 3 commits intomainfrom
APLT-216
Jan 6, 2026
Merged

Further revise retry logic to avoid stacked retry overruns#2177
cmgrote merged 3 commits intomainfrom
APLT-216

Conversation

@cmgrote
Copy link
Collaborator
@cmgrote cmgrote commented Jan 6, 2026

Note

Overhauls HTTP retry behavior to prevent stacked overruns and improve clarity and control.

  • Introduces RetryDecision enum; shouldRetry now returns a decision instead of boolean
  • sendWithRetries uses decisions to increment or reset retry, tracks separate attempts, and centralizes sleeping/backoff
  • Adds proper 429 handling: parse Retry-After, apply waits via GlobalRateLimiter, and fall back to exponential backoff
  • Supports short vs long delays via sleepTime(..., longTime) and waitLongTime
  • Removes inline sleeps from shouldRetry and consolidates logging
  • Dependency: bumps awssdk to 2.41.2 in libs.versions.toml

Written by Cursor Bugbot for commit 0d9337f. This will update automatically on new commits. Configure here.

Signed-off-by: Chris (He/Him) <cgrote@gmail.com>
@cmgrote cmgrote added the bug Something isn't working label Jan 6, 2026
@cmgrote cmgrote enabled auto-merge January 6, 2026 12:16
Signed-off-by: Chris (He/Him) <cgrote@gmail.com>
…ial back-off

Signed-off-by: Chris (He/Him) <cgrote@gmail.com>
@cmgrote cmgrote merged commit 32a19ec into main Jan 6, 2026
6 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

0