surafel58 commented on issue #11911:
URL: https://github.com/apache/seatunnel/issues/11911#issuecomment-5422212030

   I would like to work on this, starting now that #11827 has merged. Plan as 
discussed above: add bounded retry with backoff inside flush() so all callers 
benefit (batch_size, timer flush, prepareCommit, close); retry transport errors 
and retryable statuses (5xx and 429) while failing fast on other 4xx; treat a 
400 whose body indicates a duplicate or out-of-order sample as a tolerated 
delivery; reuse an existing retry helper (RetryUtils) if it fits; expose max 
attempts and backoff (initial + cap) as sink options with conservative 
defaults, bounded so a genuine outage still fails after exhaustion; and add 
docs (EN and ZH) plus tests for transient-failure-then-success and 
retry-exhaustion. PR to follow.


-- 
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]

Reply via email to