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

Jump to bottom

TransportClient.connectedNodes should contain up to date node info when using SimpleNodeSampler #4162

Closed
bleskes opened this issue Nov 13, 2013 · 0 comments
Closed

TransportClient.connectedNodes should contain up to date node info when using SimpleNodeSampler #4162

bleskes opened this issue Nov 13, 2013 · 0 comments
Assignees
@bleskes
Labels
>enhancement v0.90.8 v1.0.0.Beta2

Comments

@bleskes
Copy link
Contributor

bleskes commented Nov 13, 2013

The transport client exposes a list of nodes it's connected to via the connectedNodes property. If sniffing is turned off (which is default), that list is not updated to contain actually node information, like ID, name, attributes etc.

Having that information is handy and can avoid confusing people which verify the properties of the nodes they are connected to.
@ghost ghost assigned bleskes Nov 13, 2013
bleskes added a commit that referenced this issue Nov 13, 2013
@bleskes
Make SimpleNodeSampler populate the list of connected nodes using the…
db77949 
… information returned from the cluster

This is to allow people to introspect things like data settings and attributes. Also makes it consistent with the sniff sampler.

Closes #4162
@tkurki tkurki mentioned this issue Feb 20, 2014
Closed
@bleskes bleskes mentioned this issue Apr 7, 2014
Closed
@clintongormley clintongormley mentioned this issue May 22, 2014
Closed
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@bleskes
Make SimpleNodeSampler populate the list of connected nodes using the…
b86b471 
… information returned from the cluster

This is to allow people to introspect things like data settings and attributes. Also makes it consistent with the sniff sampler.

Closes elastic#4162
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bleskes bleskes

Labels
>enhancement v0.90.8 v1.0.0.Beta2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bleskes