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

Poor document table rendering performance #12665

Closed
2 tasks
weltenwort opened this issue Jul 5, 2017 · 5 comments
Closed
2 tasks

Poor document table rendering performance #12665

weltenwort opened this issue Jul 5, 2017 · 5 comments
Labels
Feature:Discover Discover Application performance release_note:enhancement Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL.

Comments

@weltenwort
Copy link
Member

weltenwort commented Jul 5, 2017

The current rendering of the document table takes several seconds for just 500 documents, which makes its use with larger datasets impractical.

⚠️ The measurements and goals in this issue are a work in progress and will be expanded over time.

Progress

  • measure current performance
  • define performance targets

relates to #10682

@foxylion
Copy link

foxylion commented Jul 5, 2017

In our use case the fetching of data is the most time consuming part of the rendering.

In an ideal case I think it should only fetch the first X records and fetch later on more records.
The first query should not include all possible records, but only the first 50 or so.
Elasticsearch does support continuing the search after the position of the last record (e.g. the 50th):
https://www.elastic.co/guide/en/elasticsearch/reference/master/search-request-search-after.html

@Bargs
Copy link
Contributor

Bargs commented Jul 5, 2017

Related #11457

@timroes timroes added Team:Visualizations Visualization editors, elastic-charts and infrastructure and removed Team:Visualizations Visualization editors, elastic-charts and infrastructure :Discovery labels Sep 16, 2018
@timroes timroes added Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. and removed Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Aug 31, 2021
@lizozom
Copy link
Contributor

lizozom commented Nov 2, 2021

@timroes @kertal can this be closed? I think this may be resolved.

@weltenwort weltenwort removed their assignment Nov 2, 2021
@kertal
Copy link
Member

kertal commented Nov 3, 2021

I think this can be closed since it's referring to the classic table, and it seems it didn't render in 50 patches back in the days? the upcoming new table is virtualizing scrolling anyway. so this issue is news from the last century and should therefore be closed

@timroes
Copy link
Contributor

timroes commented Nov 4, 2021

Also the original issue is 4 years old, so I'd say we're closing it and in case some more specific issues come up we'll open individual more specific issues for those.

@timroes timroes closed this as completed Nov 4, 2021
@lizozom lizozom changed the title Poor document table rendering performance Poor document table rendering performance Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Discover Discover Application performance release_note:enhancement Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL.
Projects
None yet
Development

No branches or pull requests

6 participants