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

Quarkus Vert.x GRPC client hangs on big messages. #44852

Open
alex-pumpkin opened this issue Dec 1, 2024 · 3 comments
Open

Quarkus Vert.x GRPC client hangs on big messages. #44852

alex-pumpkin opened this issue Dec 1, 2024 · 3 comments
Labels
area/grpc gRPC area/vertx kind/bug Something isn't working

Comments

@alex-pumpkin
Copy link
Contributor

Describe the bug

After resolving this vertx-grpc issue Quarkus GRPC client hangs on big messages.

Expected behavior

The client should produce some kind of error.

Actual behavior

It hangs forever without completion.

How to Reproduce?

I have a small reproducer with test failing on timeout https://github.com/alex-pumpkin/quarkus-grpc-client-hangs-reproducer

Output of uname -a or ver

Linux ap 5.15.0-121-generic #131-Ubuntu SMP Fri Aug 9 08:29:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Output of java -version

openjdk version "17.0.12" 2024-07-16 OpenJDK Runtime Environment (build 17.0.12+7-Ubuntu-1ubuntu222.04) OpenJDK 64-Bit Server VM (build 17.0.12+7-Ubuntu-1ubuntu222.04, mixed mode, sharing)

Quarkus version or git rev

3.15.2

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f)

Additional information

No response

@alex-pumpkin alex-pumpkin added the kind/bug Something isn't working label Dec 1, 2024
Copy link

quarkus-bot bot commented Dec 1, 2024

/cc @alesj (grpc), @cescoffier (grpc)

@cescoffier
Copy link
Member

@alex-pumpkin was this fixed by your recent PR?

@alex-pumpkin
Copy link
Contributor Author

alex-pumpkin commented Dec 3, 2024

@cescoffier no.
My reproducer is fixed, because default limit is 4MB now. But it still hangs if message is larger.

Vertx GRPC client just sends reset to the stream but does not emit any error.

"Legacy" client returns StatusRuntimeException with CANCELLED status when message exceeds the limit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/grpc gRPC area/vertx kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants