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

Fix cluster level dense vector stats #107962

Merged
merged 10 commits into from
Jun 18, 2024

Conversation

jimczi
Copy link
Contributor

@jimczi jimczi commented Apr 26, 2024

The cluster level dense vector stats returns the total number of dense vector indices globally including the replicas. This commit fixes the total to only include the value count of the primary indices. This change aligns with the docs stats which also reports the number of primary documents when used in cluster stats. The indices stats API still reports granular results for replicas and primaries so the information is not lost.

The cluster level dense vector stats returns the total number of dense vector indices globally including the replicas.
This commit fixes the total to only include the value count of the primary indices.
This change aligns with the docs stats which also reports the number of primary documents when used in cluster stats.
The indices stats API still reports granular results for replicas and primaries so the information is not lost.
@jimczi jimczi added >non-issue :Data Management/Stats Statistics tracking and retrieval APIs v8.15.0 labels Apr 26, 2024
Copy link

Documentation preview:

@elasticsearchmachine elasticsearchmachine added the Team:Data Management Meta label for data/management team label Apr 26, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@jimczi jimczi force-pushed the dense_vector_stats_primaries branch from 4474f7d to e21096d Compare June 18, 2024 08:12
Copy link
Member

@kderusso kderusso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with one comment

docs/reference/rest-api/common-parms.asciidoc Outdated Show resolved Hide resolved
@jimczi jimczi merged commit a6470fb into elastic:main Jun 18, 2024
15 checks passed
@jimczi jimczi deleted the dense_vector_stats_primaries branch June 18, 2024 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Stats Statistics tracking and retrieval APIs >non-issue Team:Data Management Meta label for data/management team v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants