surafel58 commented on PR #11974: URL: https://github.com/apache/seatunnel/pull/11974#issuecomment-5477814378
Thanks @DanielLeens for the careful re-verification and the approve, and for the extra idempotency / retry-storm / critical-path analysis. Good call on Issue 5 (honoring a Retry-After header on 429). I agree it makes the sink a better citizen against a receiver actively signaling backpressure. I would like to take it as a fast-follow rather than fold it into this PR: the current HttpResponse only carries the status code and body, not arbitrary response headers, so reading Retry-After means plumbing response headers through HttpResponse / HttpClientProvider in the shared connector-http-base module. That is a small but cross-module change that is cleaner in its own PR than expanding this one. I will open a follow-up issue for it and reference this thread. Everything else is addressed and CI is green, so this is ready for a committer with write access to merge whenever convenient. Thanks again. -- 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]
