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

[data.search] Remove total hits shimming (legacyHitsTotal) #186144

Open
lukasolson opened this issue Jun 12, 2024 · 1 comment
Open

[data.search] Remove total hits shimming (legacyHitsTotal) #186144

lukasolson opened this issue Jun 12, 2024 · 1 comment
Assignees
Labels
chore Feature:Search Querying infrastructure in Kibana impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort performance Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. technical debt Improvement of the software architecture and operational architecture

Comments

@lukasolson
Copy link
Member

Summary:

A long time ago in an Elastic Stack far, far away, the format that Elasticsearch returned the total number of hits in a search response changed. Prior to the change, the number of hits was always returned, and after the change, you could opt in/out to receiving the exact number of hits vs. a range.

During this period of time, we updated the data.search services to have a property, legacyHitsTotal, to preserve the existing format for the hits count. As that change was a very long time ago, and as we are trying to remove server-side processing of responses to get to a point where we can stream the response directly to the client, we need to remove this special handling. We also need to verify that consumers that are displaying the count don't have any regressions in displaying the counts (or ranges) correctly.

Related: #55974, #91742

@lukasolson lukasolson added Feature:Search Querying infrastructure in Kibana chore Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. labels Jun 12, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

@lukasolson lukasolson added loe:small Small Level of Effort impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. :DataDiscovery/fix-it-week labels Jul 8, 2024
@kertal kertal added technical debt Improvement of the software architecture and operational architecture and removed :DataDiscovery/fix-it-week labels Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Feature:Search Querying infrastructure in Kibana impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort performance Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. technical debt Improvement of the software architecture and operational architecture
Projects
None yet
Development

No branches or pull requests

3 participants