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.

Sign up for GitHub

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

Jump to bottom

REST Client Reactive: skip measuring time for SSE calls #23829

Closed
michalszynkiewicz opened this issue Feb 21, 2022 · 0 comments · Fixed by #24608
Closed

REST Client Reactive: skip measuring time for SSE calls #23829

michalszynkiewicz opened this issue Feb 21, 2022 · 0 comments · Fixed by #24608
Labels
area/rest-client area/stork kind/enhancement New feature or request
Milestone
2.9.0.CR1

Comments

@michalszynkiewicz
Copy link
Member

Description

Right now we measure time of operations for all the calls.
Measuring response times for server sent events doesn't seem to have much sense.
We should try to skip gathering time statistics for SSE

Implementation ideas

No response
geoand added a commit to geoand/quarkus that referenced this issue Mar 29, 2022
@geoand
Don't record start time when Reactive REST Client returns a Multi
110b509 
Fixes: quarkusio#23829
@geoand geoand mentioned this issue Mar 29, 2022
Merged
geoand added a commit to geoand/quarkus that referenced this issue Mar 30, 2022
@geoand
Don't record start time when Reactive REST Client returns a Multi
57cfcd7 
Fixes: quarkusio#23829
geoand added a commit that referenced this issue Mar 30, 2022
@geoand
Merge pull request #24608 from geoand/#23829
0de2c19 
Don't record start time when Reactive REST Client returns a Multi
@quarkus-bot quarkus-bot bot added this to the 2.9 - main milestone Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/rest-client area/stork kind/enhancement New feature or request
Projects
None yet
Milestone
2.9.0.CR1
Development

Successfully merging a pull request may close this issue.

Don't record start time when Reactive REST Client returns a Multi geoand/quarkus
1 participant
@michalszynkiewicz