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

[Feature Request] Support for _doc_count field in star tree to represent _doc_count as part of aggregations response #15288

Closed
bharath-techie opened this issue Aug 18, 2024 · 0 comments · Fixed by #15282
Assignees
Labels
enhancement Enhancement or improvement to existing feature or request Indexing:Performance v2.17.0

Comments

@bharath-techie
Copy link
Contributor

Is your feature request related to a problem? Please describe

Any bucketized aggregation results have _doc_count as part of response. If the user has not added any _doc_count field to account for pre-aggregations, the default behavior is _doc_count will be incremented by 1 for each doc, and if the _doc_count value is provided , then OpenSearch uses the value directly.

Describe the solution you'd like

As part of star tree indexing flow, during flush , we will always add _doc_count field to the mapping. The identity value ( when no value is provided ) for each doc will be 1 and otherwise actual _doc_count value will be taken into account.

So as part of merge or query , there will always be _doc_count field present for each doc.

Related component

Indexing:Performance

Describe alternatives you've considered

No response

Additional context

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement or improvement to existing feature or request Indexing:Performance v2.17.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants