Skip to content

Conversation

@kjsbot
Copy link

@kjsbot kjsbot commented Dec 8, 2025

Description

Currently the Trino Gateway UI does not have an option to change the timezone. This PR adds a dropdown to allow the user to select any timezone they want.

Testing

  • mvnw clean install
  • Tested the UI
Screenshot 2025-12-18 at 5 58 59 AM Screenshot 2025-12-18 at 5 59 22 AM

@cla-bot cla-bot bot added the cla-signed label Dec 8, 2025
@kjsbot kjsbot added the ui Web UI label Dec 9, 2025
@mosabua
Copy link
Member

mosabua commented Dec 9, 2025

What is the motivation for this change? I would understand if it defaults to the local timezone of the browser of the user automatically and you can set it to just display UTC as stored in the database .. but I dont understand the logic to allow selection of any timezone

@kjsbot
Copy link
Author

kjsbot commented Dec 9, 2025

What is the motivation for this change? I would understand if it defaults to the local timezone of the browser of the user automatically and you can set it to just display UTC as stored in the database .. but I dont understand the logic to allow selection of any timezone

Currently, the start time and timestamp is set to use the user's timezone. However sometimes a user might want to view the data in different timezones and allowing them to choose will provide additionally flexibility during debugging. Updating LineChartData to use epoch time instead of a formatted HH:MM string also makes the timestamps more consistent and unambiguous.

@oneonestar
Copy link
Member

I think this feature make sense for teams working across multiple timezones. Grafana and Datadog also have this feature.

@felicity3786
Copy link
Contributor

I think this feature make sense for teams working across multiple timezones. Grafana and Datadog also have this feature.

Agree seems like a valid dashboard feature.

@Peiyingy
Copy link
Member

+1 on this feature make sense. In LinkedIn, we have developers from different timezones, and we've gained benefit from this changing timezone feature. This is also a common feature for dashboard applications.

@kbhatianr
Copy link
Contributor

+1 - love adding this feature in. IMO, this feature should be utilized everywhere that we show a timestamp value:

  1. Query History => Submission Time
  2. Dashboard => Started At

@kjsbot kjsbot marked this pull request as ready for review December 11, 2025 18:29
@mosabua
Copy link
Member

mosabua commented Dec 11, 2025

Sounds like we have consensus to proceed. We talked about it also in the dev sync this week. The main thing we should make sure is that the setting is not isolated to one page but is applicable to the whole app and probably visible as switch everywhere .. and that any writing of data does not get messed up

@kjsbot kjsbot marked this pull request as draft December 13, 2025 18:23
@kjsbot kjsbot changed the title Add Dropdown to Change Timezones [DRAFT WIP]Add Dropdown to Change Timezones Dec 13, 2025
@kjsbot kjsbot changed the title [DRAFT WIP]Add Dropdown to Change Timezones Add Dropdown to Change Timezones Dec 18, 2025
@kjsbot kjsbot marked this pull request as ready for review December 18, 2025 05:24
@kjsbot kjsbot requested review from ebyhr and kbhatianr December 22, 2025 05:13
@ebyhr ebyhr requested a review from ytwp December 22, 2025 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

7 participants