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

[Lens] Unnecessary data reload after saving Lens as new #104790

Closed
tsullivan opened this issue Jul 8, 2021 · 4 comments
Closed

[Lens] Unnecessary data reload after saving Lens as new #104790

tsullivan opened this issue Jul 8, 2021 · 4 comments
Labels
Feature:Lens Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@tsullivan
Copy link
Member

tsullivan commented Jul 8, 2021

Kibana version: 7.14.0=bc1

Elasticsearch version:

Server OS version:

Browser version: Chrome 91

Browser OS version: Linux

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

Describe the bug:

After editing a Lens chart and saving it as a new chart, all the data in the UI is cleared out and gets reloaded from the search. This should not happen since the search didn't change.

This doesn't happen when saving to the same Lens.

Performance issue: in a slow network, loading data can take a long time, even if its being loaded from a search session.

Steps to reproduce:
From Dashboard:

  1. Edit a Lens from a Dashboard
  2. Click "Save as", do not choose "Add to Dashboard after saving", then Save.
  3. The data is reloaded from search, but the search has not changed since before saving.

From Visualize Editor:

  1. Create a Lens or open a saved Lens
  2. If it's new, simply add some data and Save
  3. If it's an existing, click Save and choose "Save as new Lens visualization"

Expected behavior:
The search did not change, so the data should not be reloaded from search.

Screenshots (if relevant):

Reloading the data from search causes everything on the page to flash away at 0:44:
https://user-images.githubusercontent.com/908371/124845405-88776780-df4b-11eb-9b81-57e1af87a822.mp4

@tsullivan tsullivan added bug Fixes for quality problems that affect the customer experience performance Team:Visualizations Visualization editors, elastic-charts and infrastructure Feature:Lens labels Jul 8, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@flash1293
Copy link
Contributor

Maybe we can resolve this easily once the state management refactoring is done

@flash1293 flash1293 removed the bug Fixes for quality problems that affect the customer experience label Jul 8, 2021
@flash1293
Copy link
Contributor

When fixing this, make sure switching SOs via global search still works.

@stratoula
Copy link
Contributor

This has been fixed, closing this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Lens Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

No branches or pull requests

6 participants