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

Ability to define sniffing for Client Nodes only #28155

Closed
gmoskovicz opened this issue Jan 9, 2018 · 3 comments
Closed

Ability to define sniffing for Client Nodes only #28155

gmoskovicz opened this issue Jan 9, 2018 · 3 comments
Labels
:Clients/Java Low Level REST Client Minimal dependencies Java Client for Elasticsearch >enhancement

Comments

@gmoskovicz
Copy link
Contributor

@javanna please let me know your thoughts here.

Right now if you want to allow an application to use Client Nodes only, you need to define the nodes in the connection settings for the Client. If you add sniffing, you will be able to sniff any node and you could use the sniffHosts option to sniff just client nodes from an external source.

If you want to have a dynamic pool of nodes (client only nodes) that when you add a new client node to a cluster, that node is added to the connection pool, but when you add master or data nodes, those aren't used, one of the ways that people would do that is by using an external Load Balancer.

I think that it will be great for the Java REST client to be able to have a way to sniff client-only nodes.

The current Transport Client doesn't do that, and actually just retrieves a list of data nodes. The reason the transport client excludes non-data nodes is to avoid sending search traffic to master only nodes (from the docs).

What do you think?

@gmoskovicz gmoskovicz added >enhancement :Clients/Java Low Level REST Client Minimal dependencies Java Client for Elasticsearch labels Jan 9, 2018
@dadoonet
Copy link
Member

dadoonet commented Jan 9, 2018

Or ingest nodes? I mean it could be nice to be able to filter nodes by just giving some filtering properties. Could be a rack name, a region, a tag...

@gmoskovicz
Copy link
Contributor Author

++ @dadoonet

@javanna
Copy link
Member

javanna commented Jan 16, 2018

makes sense, closing as duplicate of #21888.

@javanna javanna closed this as completed Jan 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Clients/Java Low Level REST Client Minimal dependencies Java Client for Elasticsearch >enhancement
Projects
None yet
Development

No branches or pull requests

3 participants