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

[MRESOLVER-584] Jetty transport HTTP/2 GOAWAY improvement #533

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Jul 12, 2024

Just shorten the client lifespan to connector transaction duration vs as before, for whole session duration. Also simplify the code.


https://issues.apache.org/jira/browse/MRESOLVER-584

Without "swap", just shorten the client lifespan to
connector transtion duration (vs as before for whole
session duration).

---

https://issues.apache.org/jira/browse/MRESOLVER-584
@cstamas cstamas force-pushed the MRESOLVER-584-jetty branch from 7003117 to e7b4837 Compare August 2, 2024 12:26
@cstamas cstamas changed the title [MRESOLVER-584] Jetty fix for GOAWAY [MRESOLVER-584] Jetty HTTP/2 GOAWAY improvement Aug 2, 2024
@cstamas cstamas changed the title [MRESOLVER-584] Jetty HTTP/2 GOAWAY improvement [MRESOLVER-584] Jetty transport HTTP/2 GOAWAY improvement Aug 2, 2024
@cstamas
Copy link
Member Author

cstamas commented Aug 2, 2024

@scholzi100 am about to merge this one, as I changed JIRA goal also from "fix" to "improve", which this PR does. And later we can refine things even more. This improvement is in fact very welcome simplification, and the improvement part is basically shortening the lifespan of client.

@cstamas cstamas merged commit d013a32 into apache:master Aug 2, 2024
5 checks passed
@cstamas cstamas deleted the MRESOLVER-584-jetty branch August 2, 2024 13:19
@cstamas cstamas added this to the 2.0.1 milestone Aug 2, 2024
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.

1 participant