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.

Sign up for GitHub

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

Inconsistent value types for node status/info #10404

Closed
Cidan opened this issue Apr 2, 2015 · 1 comment
Closed

Inconsistent value types for node status/info #10404

Cidan opened this issue Apr 2, 2015 · 1 comment
Labels
:Data Management/Stats Statistics tracking and retrieval APIs >enhancement good first issue low hanging fruit help wanted adoptme

Comments

@Cidan
Copy link

Cidan commented Apr 2, 2015

When viewing the following URL, http://localhost:9200/_nodes/_local, queue sizes are not consistent in types, for example:

merge: {
type: "scaling",
min: 1,
max: 4,
keep_alive: "5m",
queue_size: -1
}

versus:

get: {
type: "fixed",
min: 8,
max: 8,
queue_size: "1k"
}

The queue_size value should always be an integer, fully expanded out, ie, 1000 and not "1k". This will make monitoring these queues much simpler.
@clintongormley clintongormley added >enhancement good first issue low hanging fruit help wanted adoptme :Data Management/Stats Statistics tracking and retrieval APIs labels Apr 5, 2015
@clintongormley
Copy link

+1

xuzha pushed a commit to xuzha/elasticsearch that referenced this issue Aug 24, 2015
The queue_size value should be shown as an integer,
e252bee 
closes elastic#10404
@xuzha xuzha mentioned this issue Aug 24, 2015
Closed
@xuzha xuzha closed this as completed in f77b349 Aug 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Data Management/Stats Statistics tracking and retrieval APIs >enhancement good first issue low hanging fruit help wanted adoptme
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

The queue_size value should be shown as an integer. xuzha/elasticsearch
2 participants
@clintongormley @Cidan