-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[tableVis] additional view options #1926
Conversation
Looks great! Is it useful to show the metrics for each bucket, or could the last column just show the last non-empty bucket metric? So in the "all together now" screenshot you'd lose the 2nd column, but the final "count of documents" would contain 1 for all the records. "Show partial rows" would then imply "Calculate metrics for each bucket", so only one option would be required. Just my 2 cents though, thinking of my use cases. |
Hmm, good point. I'm concerned about the performance implications of running the metric for every bucket without the user opting-in, but I can imagine that it's the only way that partial rows would really be useful. I'll get some feedback from @rashidkpc and the gang on monday and see what they think. Thanks for helping make Kibana better! 😄 |
Awesome! |
Merge from Master and LGTM!!! |
fixes #1911
Adds two (and a half) options to the TableVis, along with many tests (which previously were only for supporting modules).
1. Show metrics for every bucket/level
default:
false
Checking this option makes the vis behave like the pie, and calculate the given metrics for each bucketting level.
2. Show partial rows
default:
false
Allow rows that are incomplete to be shown. This causes the metrics to be calculated at every level because knowing what buckets exist without knowing anything about the contents seems pretty useless. This way, when a row is incomplete, we can show data from the last bucket we were able to create.
½. Calculate metrics for every bucket/level
default:
false
This option can't actually be controlled by the user, but I'm including it in the panel to show the implications of activating something like "Show partial rows"
all together now