-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Thread locked after stress test #5443
Comments
@johnaohara have you seen anything similar? |
Which was fixed however |
@geoand Does your reproducer fail for master now? (https://github.com/geoand/quarkus-vertx-threads-blocked) |
Let me check |
Have just re-run against 1.0.0.CR1 and is working as expected |
I just ran it my reproducer against master and it worked just fine. |
I looks like there is a race condition in |
@johnaohara I've simplified my program and can reproduce it in several of my environments. My endpoint: @Path("/hello")
public class ExampleResource {
@GET
@Produces(MediaType.TEXT_PLAIN)
public String hello() {
StringBuffer buffer = new StringBuffer();
try {
Thread.sleep(10);
} catch (InterruptedException e) {
e.printStackTrace();
}
buffer.append("[");
for(int i = 0 ;i <1000;i++ ) {
buffer.append("{\"id\":1,\"name\":\"tom\",\"sex\":\"f\",\"testCode\":\"12345\"},");
}
buffer.append("{\"id\":1,\"name\":\"tom\",\"sex\":\"f\",\"testCode\":\"12345\"} ]");
return buffer.toString();
}
} Run wrk
When the wrk is running, the endpoint can be accessed through the browser.
source code |
@yuhaibohotmail thanks a lot for the reproducer, I will use it to test a potential fix. |
@yuhaibohotmail what hardware are you running on? I am not reproducing the error on my local machine, but will also test in our lab |
I can re-create now |
@johnaohara Can you reproduce it now? I tested it in two environments.
|
@yuhaibohotmail I have your test running without hanging the application. Now waiting on CI and approval for the PR. Thanks |
@yuhaibohotmail #5451 fixes this issue and will be included in the next release. Thank you for your help |
Describe the bug
(Describe the problem clearly and concisely.)
I did a simple stress test,then the program keeps giving alarm log.
After the test ,All requests are unresponsive .
jstack log:
Expected behavior
(Describe the expected behavior clearly and concisely.)
Actual behavior
(Describe the actual behavior clearly and concisely.)
To Reproduce
Steps to reproduce the behavior:
wrk -t12 -c2000 -d30s http://10.32.1.***:8888/rest/students/
Configuration
Screenshots
(If applicable, add screenshots to help explain your problem.)
Environment (please complete the following information):
Output of
uname -a
orver
:Linux ***** 3.10.0-862.3.3.el7.x86_64 Switch to the Maven distributed copy of the SubstrateVM annotations #1 SMP Fri Jun 15 04:15:27 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Output of
java -version
:java version "1.8.0_201"
Java(TM) SE Runtime Environment (build 1.8.0_201-b09)
Java HotSpot(TM) 64-Bit Server VM (build 25.201-b09, mixed mode)
GraalVM version (if different from Java):
Quarkus version or git rev: 1.0.0.CR1
Additional context
The full jstack dump info
jstack-l.txt
The text was updated successfully, but these errors were encountered: