Skip to content

fixes error with saving hidden metrics#10220

Merged
ppisljar merged 2 commits intoelastic:masterfrom
ppisljar:fix/saveHiddenMetrics
Feb 16, 2017
Merged

fixes error with saving hidden metrics#10220
ppisljar merged 2 commits intoelastic:masterfrom
ppisljar:fix/saveHiddenMetrics

Conversation

@ppisljar
Copy link
Copy Markdown
Contributor

@ppisljar ppisljar commented Feb 7, 2017

fixes #9156
when opening a saved visualization with hidden metrics they would show up as if they were not hidden.

@ppisljar ppisljar added the review label Feb 7, 2017
@tbragin tbragin added the Feature:Visualizations Generic visualization features (in case no more specific feature label is available) label Feb 7, 2017
@thomasneirynck thomasneirynck self-assigned this Feb 9, 2017
Copy link
Copy Markdown
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI is failing. Probably related to dashboard which is failing now on master.

@ppisljar ppisljar force-pushed the fix/saveHiddenMetrics branch from dbaba32 to ab09933 Compare February 10, 2017 08:44
@markharwood
Copy link
Copy Markdown
Contributor

Residual issue here - when following these steps:

  1. Create new pie chart and 2 terms aggs (one for country one for status field)
  2. Disable the country agg
  3. Run query
  4. Save

Pre, the patch, step 4 re-opens the viz with the country agg disabled but country data still showing in the tab:
9156_pre_patch_-_kibana

After applying this patch there is now a different error in step 4 - no data is shown at all in the pie chart after saving :

9156_post_patch_-_kibana

To reveal data I have to:

  1. toggle country agg on
  2. run query
  3. toggle country agg off
  4. run query

@ppisljar
Copy link
Copy Markdown
Contributor Author

great catch @markharwood !

@ppisljar ppisljar force-pushed the fix/saveHiddenMetrics branch from ab09933 to a8cd01e Compare February 13, 2017 12:59
@ppisljar
Copy link
Copy Markdown
Contributor Author

@markharwood @thomasneirynck i updated this, please take another look.

@markharwood
Copy link
Copy Markdown
Contributor

Works OK here for me now.

Copy link
Copy Markdown
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works. The hidden metrics are hidden after reopening a visualization with hidden metrics.

There seems to be a small issue, happening right after saving the Viz. The column with the hidden metrics remains visible, but the values are blanked out. This seems particular to Data Table though.

  1. Make a Data Table vizualization. Select 3 metrics
  2. save this

image

  1. hide a metric

image

  1. save the visualization again. The column with the hidden metric remains visible, but contains no data.

image

@ppisljar ppisljar merged commit 686a709 into elastic:master Feb 16, 2017
elastic-jasper added a commit that referenced this pull request Feb 16, 2017
Backports PR #10220

**Commit 1:**
fixing pie chart tooltip custom label bug

* Original sha: a8cd01e
* Authored by ppisljar <peter.pisljar@gmail.com> on 2017-02-07T12:18:45Z

**Commit 2:**
fixing the bug thomas found

* Original sha: eef0757
* Authored by ppisljar <peter.pisljar@gmail.com> on 2017-02-13T19:43:27Z
ppisljar pushed a commit that referenced this pull request Feb 16, 2017
Backports PR #10220

**Commit 1:**
fixing pie chart tooltip custom label bug

* Original sha: a8cd01e
* Authored by ppisljar <peter.pisljar@gmail.com> on 2017-02-07T12:18:45Z

**Commit 2:**
fixing the bug thomas found

* Original sha: eef0757
* Authored by ppisljar <peter.pisljar@gmail.com> on 2017-02-13T19:43:27Z
@ppisljar ppisljar deleted the fix/saveHiddenMetrics branch February 16, 2017 08:46
@ppisljar ppisljar restored the fix/saveHiddenMetrics branch September 26, 2018 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Visualizations Generic visualization features (in case no more specific feature label is available) review v5.4.0 v6.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hidden metrics/buckets show up in visualization after save

4 participants