Skip to content

Display time in local time#18455

Merged
FrankChen021 merged 13 commits intoapache:masterfrom
GabrielCWT:web-console-show-local-time
Oct 10, 2025
Merged

Display time in local time#18455
FrankChen021 merged 13 commits intoapache:masterfrom
GabrielCWT:web-console-show-local-time

Conversation

@GabrielCWT
Copy link
Copy Markdown
Contributor

@GabrielCWT GabrielCWT commented Aug 29, 2025

Closes #18453

Display local time allow users to more easily interpret times. The time being displayed (UTC or local) will be configurable via a new tab in settings. The configs will be stored locally in local storage as I felt that it should be on a per user basis.

The url will maintain the query params in UTC time but the FE will support this and convert the filter into a local time filter if the user has the setting turned on.

The query param will be interpreted as a literal in the case where the operator is ~ since it is meant to be a LIKE query. i.e. 2025-09-05T11 will be treated as such and not be converted.

Below are the new column formats
Screenshot 2025-08-29 at 4 11 31 PM
Screenshot 2025-09-05 at 3 07 42 PM
Screenshot 2025-09-05 at 3 07 30 PM

Below is the new "Web Console" Tab for configs and the modal for it.
Screenshot 2025-09-05 at 2 21 10 PM
Screenshot 2025-09-05 at 2 21 38 PM

@FrankChen021
Copy link
Copy Markdown
Member

@vogievetsky I think this change is reasonable. What do you think?

@kfaraz
Copy link
Copy Markdown
Contributor

kfaraz commented Aug 31, 2025

Given that we have always had the time displayed in UTC, this should be a config in the web-console that can be changed in the settings (top right corner).

@GabrielCWT
Copy link
Copy Markdown
Contributor Author

Given that we have always had the time displayed in UTC, this should be a config in the web-console that can be changed in the settings (top right corner).

Agreed. I have created a new tab for "Web Console Configurations", currently it will just be this local time config

@cryptoe cryptoe requested a review from vogievetsky September 3, 2025 15:13
@vogievetsky
Copy link
Copy Markdown
Contributor

Just saw this. Sounds like a cool idea. Will review in detail later today or tomorrow.

Copy link
Copy Markdown
Member

@FrankChen021 FrankChen021 left a comment

Choose a reason for hiding this comment

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

LGTM

@GabrielCWT
Copy link
Copy Markdown
Contributor Author

Hi @vogievetsky could you take a look at this PR when you have the time? Thanks!

@FrankChen021 FrankChen021 added this to the 35.0.0 milestone Oct 8, 2025
@FrankChen021
Copy link
Copy Markdown
Member

Hi @vogievetsky It's better that this can be merged into 35.
If you're too busy to review this change, I will merge it first.

@FrankChen021 FrankChen021 merged commit 42c7b06 into apache:master Oct 10, 2025
10 checks passed
cecemei pushed a commit to cecemei/druid that referenced this pull request Oct 16, 2025
* Display time in local time

* Add config for displaying local time

* Format code

* Fix date format

* Support local time in segments tab

* Update menu icon

* Fix bugs

* Fix formatting

* Update snapshots

* Clean up files

* Update snapshots

* Format lastCompletedTaskTime and blacklistedUntil
cecemei pushed a commit that referenced this pull request Oct 17, 2025
* Display time in local time

* Add config for displaying local time

* Format code

* Fix date format

* Support local time in segments tab

* Update menu icon

* Fix bugs

* Fix formatting

* Update snapshots

* Clean up files

* Update snapshots

* Format lastCompletedTaskTime and blacklistedUntil
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.

Display time on web console in local timezone

4 participants