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

Introduce elasticsearch nio library jar #27802

Closed
Tim-Brooks opened this issue Dec 14, 2017 · 1 comment
Closed

Introduce elasticsearch nio library jar #27802

Tim-Brooks opened this issue Dec 14, 2017 · 1 comment
Assignees
Labels
:Distributed/Network Http and internode communication implementations >enhancement v7.0.0-beta1

Comments

@Tim-Brooks
Copy link
Contributor

Currently our nio work lives in :test:framwork. This allows us to use nio as the basis for a test TcpTransport implementation in internal cluser integration tests. This verification is very valuable and we do not want to lose it. However, we need the base nio facilities to be available for creating a production TcpTransport implementation (in a plugin/module) and a production http transport implementation. This means that the base nio facilities need to be available elsewhere (while remaining available in :test:framwork).

In order to do this, we need to introduce a nio library jar that provides the building blocks for creating transports based on nio. Ideally, this jar will not depend on elasticsearch:core as we would like to keep open the possibility that this nio work could eventually drive the rest client.

This is related to #27260.

@Tim-Brooks Tim-Brooks added :Distributed/Network Http and internode communication implementations >enhancement v7.0.0 labels Dec 14, 2017
@Tim-Brooks Tim-Brooks self-assigned this Dec 14, 2017
Tim-Brooks added a commit that referenced this issue Dec 20, 2017
This is related to #27802. This commit adds a jar called
elasticsearch-nio that contains the base nio classes that will be used
for the tcp nio transport and eventually the http nio transport.

The jar does not depend on elasticsearch:core, so all references to core
have been removed.
@Tim-Brooks
Copy link
Contributor Author

Closed by 06b3130.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed/Network Http and internode communication implementations >enhancement v7.0.0-beta1
Projects
None yet
Development

No branches or pull requests

2 participants