Skip to content
This repository was archived by the owner on Jun 24, 2022. It is now read-only.

Disable the Netty recycler #279

Merged
merged 1 commit into from
Apr 6, 2017
Merged

Disable the Netty recycler #279

merged 1 commit into from
Apr 6, 2017

Conversation

cinience
Copy link
Contributor

@elasticmachine
Copy link
Collaborator

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run.

1 similar comment
@elasticmachine
Copy link
Collaborator

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run.

@jpcarey
Copy link
Contributor

jpcarey commented Apr 1, 2017

The linked PR also has
+-Dio.netty.allocator.type=unpooled
Which seems missing in this.

@jasontedor
Copy link
Member

@jpcarey That change was reverted in elastic/elasticsearch@f1ee224.

@jakommo
Copy link
Contributor

jakommo commented Apr 6, 2017

@cinience mind signing the CLA so I can merge this?

@cinience
Copy link
Contributor Author

cinience commented Apr 6, 2017

@jakommo done

@jakommo
Copy link
Contributor

jakommo commented Apr 6, 2017

Jenkins, test it!

@jakommo
Copy link
Contributor

jakommo commented Apr 6, 2017

Thanks @cinience

@jakommo
Copy link
Contributor

jakommo commented Apr 6, 2017

The failed tests seem not related to this, so I'm merging.
Thanks for contributing @cinience

@jakommo jakommo merged commit 5a3805b into elastic:master Apr 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants