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

Indices API returns wrong search stats groups in ES 1.3 #7644

Closed
belevian opened this issue Sep 8, 2014 · 0 comments
Closed

Indices API returns wrong search stats groups in ES 1.3 #7644

belevian opened this issue Sep 8, 2014 · 0 comments
Assignees
@alexksikes
Labels
>bug help wanted adoptme

Comments

@belevian
Copy link
Contributor

belevian commented Sep 8, 2014

Hi,

We encountered a problem with the search stats groups in Indices API.

The query_count for stats groups is false at shards level. It make fine query monitoring at shards/nodes level impossible.

I have created a curl recreation. It has been tested with ES 1.3.2 on my recent MBP.

https://gist.github.com/belevian/46026b88500d264fbc66

Thanks in advance to fix this problem.

Do not hesitate to ask if you need more informations.

Regards

Benjamin
@alexksikes alexksikes self-assigned this Sep 25, 2014
alexksikes added a commit to alexksikes/elasticsearch that referenced this issue Dec 15, 2014
@alexksikes
Indices API: Fix wrong search stats groups
9f1abd8 
This provides a fix to issue elastic#7644.
@alexksikes alexksikes mentioned this issue Dec 15, 2014
Closed
alexksikes added a commit that referenced this issue Dec 16, 2014
@alexksikes
Indices API: Fix wrong search stats groups
086ac9c 
This provides a fix to issue #7644. A new Stats object must be created, and
not a reference to the retrieved stats, before we can add stats to it.
Otherwise, we would keep on adding to the same object on subsequent calls to
IndicesStatsResponse#getPrimaries() or IndicesStatsResponse#getTotal().

Closes #7644 and #8950
alexksikes added a commit that referenced this issue Jan 5, 2015
@alexksikes
Indices API: Fix wrong search stats groups
cfbc29d 
This provides a fix to issue #7644. A new Stats object must be created, and
not a reference to the retrieved stats, before we can add stats to it.
Otherwise, we would keep on adding to the same object on subsequent calls to
IndicesStatsResponse#getPrimaries() or IndicesStatsResponse#getTotal().

Closes #7644 and #8950
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@alexksikes
Indices API: Fix wrong search stats groups
3972127 
This provides a fix to issue elastic#7644. A new Stats object must be created, and
not a reference to the retrieved stats, before we can add stats to it.
Otherwise, we would keep on adding to the same object on subsequent calls to
IndicesStatsResponse#getPrimaries() or IndicesStatsResponse#getTotal().

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

@alexksikes alexksikes

Labels
>bug help wanted adoptme
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@alexksikes @clintongormley @belevian