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

Logs Panel: Column selection for experimental table visualization in explore #76983

Merged

Conversation

gtk-grafana
Copy link
Contributor

@gtk-grafana gtk-grafana commented Oct 23, 2023

What is this feature?
Experimental
New UI for selecting columns in table visualization (under logsExploreTableVisualisation feature flag) introduced in #71120.

Who is this feature for?
Internal users and early adopters that use logs in explore.

Special notes for your reviewer:
Took the work @svennergr started and added a HoC component (LogsTableWrap) to receive the dataframe and create a filtered copy of that data frame based on the current state of the filters, which is passed into the dataframe Table component. The filtering and extracting of fields (columns) is handled by the organizeFields and extractFields transformations.

Tested with cloudwatch, loki, and elastic

image image

Please check that:

  • It works as expected from a user's perspective.
  • Dataframe compatible.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

gtk-grafana and others added 30 commits October 12, 2023 15:57
…hooking up hardcoded panel state to explore url and syncing
… reflected in explore url, and initial state from url is populating
…e with fewer or more labels would render additional unwanted columns before getting cleaned up on the next render. Also documenting and cleaning up
* Sync panel state with explore URL state
…-state' of https://github.com/grafana/grafana into gtk-grafana/features/logs/table-viz-explore-poc__no-url-state
@gtk-grafana gtk-grafana added add to changelog and removed no-changelog Skip including change in changelog/release notes labels Oct 25, 2023
@gtk-grafana gtk-grafana changed the title Logs Panel: Column selection UI in table visualization Logs Panel: Column selection for experimental table visualization in explore Oct 25, 2023
Copy link
Member

@ivanahuckova ivanahuckova left a comment

Choose a reason for hiding this comment

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

Nice! LGTM.

* Sync explore URL state with logs panel state in explore
@gtk-grafana gtk-grafana requested review from a team as code owners October 26, 2023 16:18
@gtk-grafana gtk-grafana requested review from Clarity-89, eledobleefe and leventebalogh and removed request for a team October 26, 2023 16:18
@gtk-grafana gtk-grafana merged commit 05376a9 into main Oct 26, 2023
21 checks passed
@gtk-grafana gtk-grafana deleted the gtk-grafana/features/logs/table-viz-explore__url-sync branch October 26, 2023 17:06
ssama88 pushed a commit to ssama88/grafana that referenced this pull request Oct 31, 2023
…explore (grafana#76983)

* New experimental table customization for logs in explore
* Logs Panel: Explore url sync for table visualization (grafana#76980)
* Sync explore URL state with logs panel state in explore
@aangelisc aangelisc modified the milestones: 10.3.x, 10.2.3 Dec 21, 2023
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.

None yet

4 participants