Skip to content

Changing name of shards field in node/stats api to shard_stats #78531#78803

Merged
masseyke merged 2 commits intoelastic:7.15from
masseyke:fix/nodes-stats-two-shards-7.15
Oct 6, 2021
Merged

Changing name of shards field in node/stats api to shard_stats #78531#78803
masseyke merged 2 commits intoelastic:7.15from
masseyke:fix/nodes-stats-two-shards-7.15

Conversation

@masseyke
Copy link
Copy Markdown
Member

@masseyke masseyke commented Oct 6, 2021

If the _nodes/stats API received a level=shards request parameter, then the response would have two "shards" fields,
which would cause problems with json parsers. This commit renames the "shards" field that currently only contains
"total_count" to "shard_stats".
Relates #78311 #75433 #78531

…ic#78531)

If the _nodes/stats API received a level=shards request parameter, then the response would have two "shards" fields,
which would cause problems with json parsers. This commit renames the "shards" field that currently only contains
"total_count" to "shard_stats".
Relates elastic#78311 elastic#75433
@masseyke masseyke added the :Core/Infra/Stats Statistics tracking and retrieval APIs label Oct 6, 2021
@masseyke masseyke requested review from dakrone and jakelandis October 6, 2021 22:54
@elasticmachine elasticmachine added the Team:Data Management (obsolete) DO NOT USE. This team no longer exists. label Oct 6, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

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

@masseyke masseyke merged commit 7e58aa6 into elastic:7.15 Oct 6, 2021
@masseyke masseyke deleted the fix/nodes-stats-two-shards-7.15 branch November 2, 2021 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport :Core/Infra/Stats Statistics tracking and retrieval APIs Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v7.15.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants