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

Using quarkus-rest-client-reactive-jackson on rest-client-quickstart blocks when Uni is returned #16148

Closed
geoand opened this issue Mar 31, 2021 · 1 comment · Fixed by #16167
Closed

Using quarkus-rest-client-reactive-jackson on rest-client-quickstart blocks when Uni is returned #16148

geoand opened this issue Mar 31, 2021 · 1 comment · Fixed by #16167
Assignees
@geoand
Labels
area/rest-client kind/bug Something isn't working
Milestone
1.13.1.Final

Comments

@geoand
Copy link
Contributor

geoand commented Mar 31, 2021 •
edited
Loading

Describe the bug

When taking https://github.com/quarkusio/quarkus-quickstarts/tree/main/rest-client-quickstart and using quarkus-resteasy-reactive-jackson and quarkus-rest-client-reactive-jackson instead of the original RESTEasy and rest-client dependencies (and also substituting @PathParam for @RestPath), then running

mvn test -Dtest=CountriesResourceTest#testCountryNameMutinyEndpoint

results in a timeout exception that seems to be due to the underlying implementation of the client attempting to use blocking IO
@geoand geoand added kind/bug Something isn't working area/rest-client labels Mar 31, 2021
@geoand geoand self-assigned this Mar 31, 2021
@geoand geoand changed the title Using quarkus-rest-client-reactive-jackson on rest-client-quickstart blocks when Uni is returned Using quarkus-rest-client-reactive-jackson on rest-client-quickstart blocks when ComletionStage is returned Mar 31, 2021
@geoand geoand changed the title Using quarkus-rest-client-reactive-jackson on rest-client-quickstart blocks when ComletionStage is returned Using quarkus-rest-client-reactive-jackson on rest-client-quickstart blocks when Uni is returned Mar 31, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Mar 31, 2021

/cc @michalszynkiewicz

@snowdrop-bot snowdrop-bot mentioned this issue Mar 31, 2021
Closed
geoand added a commit to geoand/quarkus that referenced this issue Apr 1, 2021
@geoand
Fix Uni response type handling in quarkus-rest-client-reactive
8977265 
Uni was not being treated as an async return type resulting in
the client making blocking calls and trying to deserialize the
result into a Uni

Fixes: quarkusio#16148
@geoand geoand mentioned this issue Apr 1, 2021
Merged
michalszynkiewicz added a commit that referenced this issue Apr 1, 2021
@michalszynkiewicz
Merge pull request #16167 from geoand/#16148
dc40c52 
Fix Uni response type handling in quarkus-rest-client-reactive
@quarkus-bot quarkus-bot bot added this to the 1.14 - main milestone Apr 1, 2021
@gsmet gsmet modified the milestones: 1.14 - main, 1.13.1.Final Apr 3, 2021
gsmet pushed a commit to gsmet/quarkus that referenced this issue Apr 3, 2021
@geoand @gsmet
Fix Uni response type handling in quarkus-rest-client-reactive
a9e2c2c 
Uni was not being treated as an async return type resulting in
the client making blocking calls and trying to deserialize the
result into a Uni

Fixes: quarkusio#16148
(cherry picked from commit 8977265)
juazugas pushed a commit to juazugas/quarkus that referenced this issue Sep 30, 2021
@geoand @juazugas
Fix Uni response type handling in quarkus-rest-client-reactive
fbcba8e 
Uni was not being treated as an async return type resulting in
the client making blocking calls and trying to deserialize the
result into a Uni

Fixes: quarkusio#16148
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@geoand geoand

Labels
area/rest-client kind/bug Something isn't working
Projects
None yet
Milestone
1.13.1.Final
Development

Successfully merging a pull request may close this issue.

Fix Uni response type handling in quarkus-rest-client-reactive geoand/quarkus
2 participants
@gsmet @geoand