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

Drop ClusterStateHealth#indices when unnecessary #90631

Open
Tracked by #77466
DaveCTurner opened this issue Oct 4, 2022 · 2 comments · May be fixed by #100879
Open
Tracked by #77466

Drop ClusterStateHealth#indices when unnecessary #90631

DaveCTurner opened this issue Oct 4, 2022 · 2 comments · May be fixed by #100879
Labels
>bug :Data Management/Stats Statistics tracking and retrieval APIs Team:Data Management Meta label for data/management team

Comments

@DaveCTurner
Copy link
Contributor

ClusterStateHealth#indices might be many MBs of data in a large cluster but it's only used by GET _cat/indices and GET _cluster/health?level={indices|shards} so it could reasonably be left empty in all other cases. Similarly ClusterIndexHealth#shards is only used by GET _cluster/health?level=shards and could be omitted elsewhere, giving at least a 2× saving in those cases too.

Relates #77466.

@DaveCTurner DaveCTurner added >bug :Data Management/Stats Statistics tracking and retrieval APIs labels Oct 4, 2022
@elasticsearchmachine elasticsearchmachine added the Team:Data Management Meta label for data/management team label Oct 4, 2022
@elasticsearchmachine
Copy link
Collaborator

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

@NEUpanning
Copy link
Contributor

I've opened a pull request for this issue in #100879

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Data Management/Stats Statistics tracking and retrieval APIs Team:Data Management Meta label for data/management team
Projects
None yet
3 participants