Skip to content

Display time in local time - #18455

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

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

Conversation

@GabrielCWT

@GabrielCWT GabrielCWT commented Aug 29, 2025

Copy link
Copy Markdown
Contributor

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

kfaraz commented Aug 31, 2025

Copy link
Copy Markdown
Contributor

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.

@FrankChen021 FrankChen021 left a comment

Copy link
Copy Markdown
Member

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
riovic918data pushed a commit to riovic918data/druid that referenced this pull request Jun 12, 2026
* 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