We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
time
getTime
The GetStats class serializes what should be the time field as getTime instead.
GetStats
This issue was introduced in a refactoring from nearly twelve years ago (ie. very pre-fork) where a find & replace appears to have gone awry: cc83c2f#diff-f7abcc8f5eb3ec75bb9be11b7fe88baa0bcedf62c3c481552a0e98ceb7125a27R124
This means that when getting stats for an index, the "human" variant of time_in_millis becomes getTime rather than just time as expected.
time_in_millis
Other
PUT /movies
GET /movies/_stats?human&filter_path=_all.primaries.get
{ "_all": { "primaries": { "get": { "total": 0, "getTime": "0s", "time_in_millis": 0, "exists_total": 0, "exists_time": "0s", "exists_time_in_millis": 0, "missing_total": 0, "missing_time": "0s", "missing_time_in_millis": 0, "current": 0 } } } }
The getTime field should be named simply time to align with all other "human" _in_millis pairings.
_in_millis
Screenshots
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Describe the bug
The
GetStats
class serializes what should be thetime
field asgetTime
instead.This issue was introduced in a refactoring from nearly twelve years ago (ie. very pre-fork) where a find & replace appears to have gone awry: cc83c2f#diff-f7abcc8f5eb3ec75bb9be11b7fe88baa0bcedf62c3c481552a0e98ceb7125a27R124
This means that when getting stats for an index, the "human" variant of
time_in_millis
becomesgetTime
rather than justtime
as expected.Related component
Other
To Reproduce
PUT /movies
GET /movies/_stats?human&filter_path=_all.primaries.get
Expected behavior
The
getTime
field should be named simplytime
to align with all other "human"_in_millis
pairings.Additional Details
Screenshots
The text was updated successfully, but these errors were encountered: