Skip to content

HTTP182: Add retry logic with delivery guarantees, retry all failed requests#185

Open
Jonathan Lehto (jonathanlehto) wants to merge 3 commits into
getindata:mainfrom
jonathanlehto:http182-2
Open

HTTP182: Add retry logic with delivery guarantees, retry all failed requests#185
Jonathan Lehto (jonathanlehto) wants to merge 3 commits into
getindata:mainfrom
jonathanlehto:http182-2

Conversation

@jonathanlehto
Copy link
Copy Markdown

@jonathanlehto Jonathan Lehto (jonathanlehto) commented Oct 31, 2025

Description

Add retry logic with delivery guarantees, retry all failed requests

Resolves 182

PR Checklist

@jonathanlehto
Copy link
Copy Markdown
Author

David Radley (@davidradl) , I have a http182 mr as well if we want to go ahead with something like #184 as that will conflict with this branch. Thanks!

@davidradl
Copy link
Copy Markdown
Contributor

Jonathan Lehto (@jonathanlehto) Sorry for the long delay; this and the other 2 PRs are capabilities that we (IBM) would like to get merged. I think the current state of the code is good enough to merge, but to be extra diligent I would like to have this ported to the Apache HTTP connector and have Ferenc review it. Are you ok to bring this code up to date here and port to the Apache HTTP connector - under 3 Apache Flink Jiras. Once it is reviewed in the apache repo we can implement any feedback and merge in both places. If you port the fix these will be recognised Apache Flink contributions. If you are no long wanting to work on this let me know and I will take over when it gets to the top of my priority list.

fyi Grzegorz Kołakowski (@grzegorz8)

@davidradl
Copy link
Copy Markdown
Contributor

Grzegorz Kołakowski (@grzegorz8) please could you run the workflow for this

@jonathanlehto
Copy link
Copy Markdown
Author

Grzegorz Kołakowski (@grzegorz8) please could you run the workflow for this

Looks like a test is failing, I will add a new commit in a moment here

@davidradl
Copy link
Copy Markdown
Contributor

Grzegorz Kołakowski (@grzegorz8) please could you run the workflow for this

@davidradl
Copy link
Copy Markdown
Contributor

Jonathan Lehto (@jonathanlehto) there is a unit test failure in the CI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants