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

EpollEventLoopGroup slow shutdown regression #9678

Closed
vietj opened this issue Oct 16, 2019 · 9 comments
Closed

EpollEventLoopGroup slow shutdown regression #9678

vietj opened this issue Oct 16, 2019 · 9 comments
Milestone

Comments

@vietj
Copy link
Contributor

vietj commented Oct 16, 2019

Expected behavior

EpollEventLoopGroup regular shutdown takes a few millis to complete.

Actual behavior

EpollEventLoopGroup takes about 1 second to shutdown. This only apply since 4.1.42.Final, it might be related to 76592db#diff-824cfcec3522343772416dd064658a83

Steps to reproduce

Shutdown an EpollEventLoopGroup and wait for its completion. We noticed it because Vert.x testsuite using Epoll on Linux takes much longer than usual (making timeout CI jobs).

Minimal yet complete reproducer code (or URL to code)

Clone https://github.com/eclipse-vertx/vert.x and run mvn test -PtestNativeTransport

Netty version

4.1.42.Final

JVM version (e.g. java -version)

OpenJDK 8

OS version (e.g. uname -a)

Linux XXX 4.4.0-93-generic #116-Ubuntu SMP Fri Aug 11 21:17:51 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

@njhill
Copy link
Member

njhill commented Oct 16, 2019

@vietj would you be able to try with the latest 4.1.43 snapshot? Some fixes have already gone in which hopefully address this.

@vietj
Copy link
Contributor Author

vietj commented Oct 16, 2019

sure, do you have snapshots published in sonatype OSS repository ?

@normanmaurer
Copy link
Member

@vietj yes

@vietj
Copy link
Contributor Author

vietj commented Oct 16, 2019

awesome will give a run soon.

@vietj
Copy link
Contributor Author

vietj commented Oct 16, 2019

assuming of course sonatype repository works :-)

@vietj
Copy link
Contributor Author

vietj commented Oct 16, 2019

it does not seem to work for me at the moment, I'll report later

@normanmaurer
Copy link
Member

@vietj let us know once you have any feedback to share

@vietj
Copy link
Contributor Author

vietj commented Oct 17, 2019

it works fine with latest 4.1..43.Final-SNAPSHOT https://travis-ci.org/vietj/vert.x/builds/599121826

@normanmaurer
Copy link
Member

Ok cool then lets close this

@normanmaurer normanmaurer added this to the 4.1.43.Final milestone Oct 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants