Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable retries of failed requests #6

Merged
merged 1 commit into from
Jul 6, 2022
Merged

Disable retries of failed requests #6

merged 1 commit into from
Jul 6, 2022

Conversation

swtwsk
Copy link
Contributor

@swtwsk swtwsk commented Jul 6, 2022

PR Checklist

@swtwsk swtwsk requested a review from kristoffSC July 6, 2022 08:27
@github-actions
Copy link
Contributor

github-actions bot commented Jul 6, 2022

File Coverage [71.72%] 🍏
HttpSinkWriter.java 71.72% 🍏
Total Project Coverage 92.69% 🍏

@swtwsk swtwsk force-pushed the http-sink-no-retry branch from 4dc2bec to d25d1cb Compare July 6, 2022 09:12
@github-actions
Copy link
Contributor

github-actions bot commented Jul 6, 2022

File Coverage [71.72%] 🍏
HttpSinkWriter.java 71.72% 🍏
Total Project Coverage 92.69% 🍏

Copy link
Collaborator

@kristoffSC kristoffSC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

@swtwsk swtwsk merged commit 699bdee into main Jul 6, 2022
@swtwsk swtwsk deleted the http-sink-no-retry branch July 6, 2022 09:22
kristoffSC added a commit that referenced this pull request Jun 26, 2023
kristoffSC added a commit that referenced this pull request Jun 27, 2023
kristoffSC added a commit that referenced this pull request Jul 4, 2023
* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink.

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #2

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #3

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #4

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #5 tests

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #6 tests

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #7 tests

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #8 tests

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #9 tests

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP - doc.

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #9 Java doc

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #10 - changes after Code review - fix typos

Signed-off-by: Krzysztof Chmielewski <[email protected]>

* HTTP-42-BatchRequest - add support for batch request processing in HTTP sink #10 - changes after Code review - fix batch split

Signed-off-by: Krzysztof Chmielewski <[email protected]>

---------

Signed-off-by: Krzysztof Chmielewski <[email protected]>
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