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

[Discover] [Unified Field List] Field stats load exceptionally slow on large data sets #151542

Open
davismcphee opened this issue Feb 17, 2023 · 3 comments
Labels
discuss Feature:Discover Discover Application Feature:UnifiedFieldList The unified field list component used by Lens & Discover impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:needs-research This issue requires some research before it can be worked on or estimated Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Graph)

Comments

@davismcphee
Copy link
Contributor

The new field stats popover seems to load exceptionally slow on large data sets. I imagine this is in large part because we're working with bigger samples rather than the 500 or fewer documents we used to, but it's quite disruptive and we should discuss if we're happy with this or can do anything to improve the UX:
field_stats_slow

One thought I had it would it be useful to fallback to the old field stats as a placeholder while the new field stats are loading? We could potentially do something like this:

  • Display the loading indicator initially, and if the stats have not loaded within X seconds, display the old field stats.
  • Show a loading indicator next to the old stats and a message along the lines of Calculated from a sample of up to 500 documents. More accurate stats are loading..
  • Replace the old stats with the new stats once they have loaded.
@davismcphee davismcphee added discuss Feature:Discover Discover Application Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Graph) Feature:UnifiedFieldList The unified field list component used by Lens & Discover labels Feb 17, 2023
@elasticmachine
Copy link
Contributor

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

@mattkime
Copy link
Contributor

@davismcphee How do you define a big data set?

@davismcphee
Copy link
Contributor Author

@mattkime I guess I don't have a hard definition, but in the GIF I originally posted I was working with the entirety of the nyc_taxis data set, which is pretty huge and probably an extreme example at 165m records. But taking a couple week chunk of that set with 5.5m records still results in significant enough loading times, approaching 5s for some, that a fallback might be helpful:
field_stats

@davismcphee davismcphee added impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:needs-research This issue requires some research before it can be worked on or estimated labels Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Feature:Discover Discover Application Feature:UnifiedFieldList The unified field list component used by Lens & Discover impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:needs-research This issue requires some research before it can be worked on or estimated Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Graph)
Projects
None yet
Development

No branches or pull requests

3 participants