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

Web console: Explore view improvements #17627

Merged
merged 2 commits into from
Jan 15, 2025

Conversation

vogievetsky
Copy link
Contributor

This PR makes several improvements to the (experimental) Explore view. Here are the highlights.

Better time chart visualization

time-chart

The time chart visualization has been reworked using code and components from the segment timeline visualization (#17508).

The time chart now supports zooming, dragging, and is smarter about granularity detection. It is also no longer using ECharts.

Helper tables on the right for quick filtering

helper tables

Added helper tables that allow for quick filtering and some additional context

Added layouts that allow visualization tiles to be shown side by side

localhost_18081_unified-console html(Docs) (2)

Visualizations can now be shown side by side in this simple layout configuration.

Also...

  • Fixed bug where non latin1 chars could not be saved in the URL
  • Updated some dependencies
  • Made the resource picker resizable
  • Added tooltip to show and copy last run query ID

Copy link
Member

@clintropolis clintropolis left a comment

Choose a reason for hiding this comment

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

nice 🤘

Pulled the PR locally to click around a bit and its pretty nice.

Had a few thoughts:

Should the helper tab should maybe be cleared when switching tables? (I seem to remember some other stale state when doing the same thing in previous times clicking around to find bugs too, but I can't recall exactly what and don't see it on the last big explore PR I reviewed).

I also wonder if the layout button could be a bit more obvious instead of tucked behind the ..., it feels maybe worth being at the top level on the bar so its existence isn't missed, took me a bit to find it.

@vogievetsky
Copy link
Contributor Author

Thank you for the review! I addressed the comments.

@cryptoe cryptoe merged commit 1db4557 into apache:master Jan 15, 2025
77 checks passed
@cryptoe cryptoe added this to the 32.0.0 milestone Jan 15, 2025
@vogievetsky vogievetsky deleted the new_explore_layout branch January 15, 2025 16:11
ashwintumma23 pushed a commit to ashwintumma23/druid that referenced this pull request Jan 20, 2025
* new Explore layout

* fix feedback comments
317brian pushed a commit to 317brian/druid that referenced this pull request Jan 28, 2025
* new Explore layout

* fix feedback comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants