Skip to content

Fix cluster level dense vector stats#107962

Merged
jimczi merged 10 commits intoelastic:mainfrom
jimczi:dense_vector_stats_primaries
Jun 18, 2024
Merged

Fix cluster level dense vector stats#107962
jimczi merged 10 commits intoelastic:mainfrom
jimczi:dense_vector_stats_primaries

Conversation

@jimczi
Copy link
Copy Markdown
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 :Core/Infra/Stats Statistics tracking and retrieval APIs v8.15.0 labels Apr 26, 2024
@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview:

@elasticsearchmachine elasticsearchmachine added the Team:Data Management (obsolete) DO NOT USE. This team no longer exists. label Apr 26, 2024
@elasticsearchmachine
Copy link
Copy Markdown
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
Copy Markdown
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

@jimczi jimczi merged commit a6470fb into elastic:main Jun 18, 2024
@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

:Core/Infra/Stats Statistics tracking and retrieval APIs >non-issue Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v8.15.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants