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

Flaky Test : DataDemandTest.testBlockingReadInADifferentThread() #7774

Closed
joakime opened this issue Mar 23, 2022 · 0 comments · Fixed by #7776
Closed

Flaky Test : DataDemandTest.testBlockingReadInADifferentThread() #7774

joakime opened this issue Mar 23, 2022 · 0 comments · Fixed by #7776
Assignees

Comments

@joakime
Copy link
Contributor

joakime commented Mar 23, 2022

Jetty version(s)
10.0.9-SNAPSHOT

Description

Running org.eclipse.jetty.http3.tests.DataDemandTest.testBlockingReadInADifferentThread()
Running DataDemandTest.testBlockingReadInADifferentThread() testBlockingReadInADifferentThread()
2022-03-22 19:19:01.529:INFO :oejs.Server:main: jetty-10.0.9-SNAPSHOT; built: 2022-03-22T18:35:23.990Z; git: 2a3547f0297877b5a802b8965e387854748cad02; jvm 17.0.2+8
2022-03-22 19:19:01.533:INFO :oejhs.HTTP3ServerConnector:main: HTTP/3+QUIC support is experimental and not suited for production use.
2022-03-22 19:19:01.657:INFO :oejus.SslContextFactory:main: x509=X509@29962b2f(mykey,h=[quic.tech],a=[],w=[]) for Server@7c7e73c5[provider=null,keyStore=file:///home/jenkins/agent/workspace/jetty.project_PR-7682/jetty-http3/http3-tests/src/test/resources/keystore.p12,trustStore=null]
2022-03-22 19:19:01.703:INFO :oejs.AbstractConnector:main: Started HTTP3ServerConnector@32507479{h3, (h3)}{0.0.0.0:49986}
2022-03-22 19:19:01.809:INFO :oejs.Server:main: Started Server@4ae2e781{STARTING}[10.0.9-SNAPSHOT,sto=0] @13553ms
2022-03-22 19:19:01.817:INFO :oejhc.HTTP3Client:main: HTTP/3+QUIC support is experimental and not suited for production use.
Exception in thread "Thread-290" java.lang.NullPointerException: Cannot invoke "org.eclipse.jetty.io.RetainableByteBuffer.hasRemaining()" because "this.buffer" is null
	at org.eclipse.jetty.http3.internal.HTTP3StreamConnection.tryReleaseBuffer(HTTP3StreamConnection.java:363)
	at org.eclipse.jetty.http3.internal.HTTP3StreamConnection.readData(HTTP3StreamConnection.java:211)
	at org.eclipse.jetty.http3.internal.HTTP3Stream.readData(HTTP3Stream.java:149)
	at org.eclipse.jetty.http3.tests.DataDemandTest$22.lambda$onRequest$0(DataDemandTest.java:461)
	at java.base/java.lang.Thread.run(Thread.java:833)
@joakime joakime added the Bug For general bugs on Jetty side label Mar 23, 2022
sbordet added a commit that referenced this issue Mar 24, 2022
…Thread().

Do not try to release the network buffer after having notified onDataAvailable()
because it can be in a race with the application trying to read data and also
trying to release the network buffer.

Signed-off-by: Simone Bordet <[email protected]>
@joakime joakime added Test flaky-test and removed Bug For general bugs on Jetty side labels Mar 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants