chengxilo opened a new pull request, #2884: URL: https://github.com/apache/iggy/pull/2884
## Which issue does this PR close? <!-- We generally require a GitHub issue to be filed for all bug fixes and enhancements. You can link an issue to this PR using the GitHub syntax. For example `Closes #123` indicates that this PR will close issue #123. --> N/A ## Rationale Upgrade retry-go to v5, since I just realized that v3 will crash when the WithAttempts() receive a big uint value, e.g. `math.MaxUint32`. Also, to align with the rust client behavior, go sdk should have infinite retry for reconnection. ## What changed? 1. retry-go version upgrade to v5. 2. Default attempts set to 0, which means infinite for `go-retry` ## Local Execution - Passed - Pre-commit hooks ran ## AI Usage <!-- If AI tools were used, please answer: 1. Which tools? (e.g., GitHub Copilot, Claude, ChatGPT) 2. Scope of usage? (e.g., autocomplete, generated functions, entire implementation) 3. How did you verify the generated code works correctly? 4. Can you explain every line of the code if asked? If no AI tools were used, write "None" or delete this section. --> None -- 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]
