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

[Reporting] CSV features from Dashboard: add "Generate CSV" option, deprecate "Download CSV" option #133669

Closed
tsullivan opened this issue Jun 6, 2022 · 2 comments
Labels
discuss impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. loe:medium Medium Level of Effort

Comments

@tsullivan
Copy link
Member

tsullivan commented Jun 6, 2022

Kibana version:
8.4.+

Elasticsearch version:

Server OS version:

Browser version:

Browser OS version:

Original install method (e.g. download page, yum, from source, etc.):

Describe the change:
Users have described performance issues with the "Download CSV" option in Dashboard. The workaround is to navigate to Discover and use the "Generate CSV" option to create a report of a saved search.

Background
The reason for the performance problems are due to using Kibana as a bridge between scan-and-scroll data from Elasticsearch and the end-user's browser. Certain things specific to "Download CSV' create weak links in the chain:

  • Elasticsearch may need a lot of time preparing a long search context due to a large number of results
  • A proxy can be in place that either forwards for Elasticsearch or forwards for Kibana. These proxies could time out requests that have high response latency.

Proposal
There already is a config setting to show/hide the "Download CSV" option: xpack.reporting.csv.enablePanelActionDownload. I propose 2 things:

  • Add a "Generate CSV" option to the share menu of the saved search dashboard panel, and create an Kibana Application Privilege for admins to allow users to use that option.
  • Log that usage of the xpack.reporting.csv.enablePanelActionDownload option is deprecated due to performance reasons.

This change will make the "workaround" for the performance problem a front-and-center solution in the user experience.

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-services (Team:AppServicesUx)

@exalate-issue-sync exalate-issue-sync bot added impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. loe:medium Medium Level of Effort labels Jun 6, 2022
@tsullivan
Copy link
Member Author

Closing this for a simpler proposal. Looks like there is little interest in deprecating "Download CSV", and we haven't seen a true general trend in performance or issues with it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. loe:medium Medium Level of Effort
Projects
None yet
Development

No branches or pull requests

2 participants