-
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
[Lens] Custom Breakdown Presentation Order #191988
Comments
Pinging @elastic/kibana-visualizations (Team:Visualizations) |
@stratoula is this chart only for ES|QL or also for the dataview mode? |
Both! right @davismcphee ? |
Scope for this issue: we need a bit more investigation to figure out how to solve this problem so we are repurposing this as a spike and time-boxing it to < 1 week |
@stratoula correct! This is needed for both data view and ES|QL mode. |
This functionality will be available in kibana when #195447 is merged. |
#195447 is merged but this can be achieved today only through Lens override functions, that are not persisted into Lens nor into a Dashboard. In the long run, probably these operations should be described in the "data processing" part of Lens and not connected to the data styling. |
@markov00: I agree that we should provide configuration controls for users to apply a custom order to their visualization breakdowns. However, I do have a few concerns about relegating that control to the color mapping interface:
Also, this related issue from Wylie seemed to indicate that there were use cases beyond just breakdowns. Personally, my initial instinct is to add any sort of sorting controls at the global visualization level (i.e. toolbar menus), if we need to account for that multi-layer example I describe above. If we don't have to account for the multi-layer example, then at the root-level of the appropriate layer dimension's configuration flyout (i.e. not within the deeper color mapping configuration) would be my preference. Thoughts? |
@MichaelMarcialis let's discuss this offline and understand what/where we can go |
What's the current state of this issue, as it appears to block #194549? thx |
Unfortunately we don't have the capacity to take this fully right now. We suggested some other workarounds that involves revisiting the way UnifiedHistogram works to unblock this in the mean time. |
we spoke about it this AM for an hour and likely need to sync with @ninoslavmiskovic and @davismcphee to figure out how important this is and which alternative path is the right one:
|
Thanks for the update @ghudgins 👍 I agree we should do a sync about it to work through some of the details and make sure we're all on the same page.
I don't want to mislead here, we can definitely make changes to Unified Histogram to support custom breakdown orders. There are issues with it currently so we intend to refactor regardless, but it wouldn't stop us from making necessary changes. I think the main issue with that approach is just that it's ES|QL only, which may be ok since we've been asked to focus primarily on ES|QL now, but of course a solution that works for all use cases would be preferred if feasible. Maybe this would work in combination with #7 to support both cases though:
I think I'm missing some context though since I'm not familiar with "how we do breakdown order in DSL for date histogram". Definitely something to explore in our sync. |
should we already use our Monday DataDiscovery sync for this? however it collides with VisEditors sync, but we had nice VisEditior folks at the start of our sync the last 2 weeks, it could become a tradition ... or we setup a seperate one |
As part of the One Discover contextual logs effort, we would like to present a histogram of documents broken down by the
log.level
field. To make this information easier to interpret, it would be ideal if we could have greater control over the breakdown presentation order, allowing us to order the legend and breakdown stacking bylog.level
severity (see example in mockup below). As I understand it, this is currently not possible on either the Lens and Elastic Charts side.Related issue: #86184
CCing @stratoula, @teresaalvarezsoler, @nickofthyme.
The text was updated successfully, but these errors were encountered: