[MINOR] Improvement(dashboard): Support display human-readable time format for app page (#1851) by maobaolong · Pull Request #2011 · apache/uniffle · GitHub
Skip to content

[MINOR] Improvement(dashboard): Support display human-readable time format for app page (#1851)#2011

Merged
zhengchenyu merged 1 commit into
apache:branch-0.9from
maobaolong:cp1851
Aug 9, 2024
Merged

[MINOR] Improvement(dashboard): Support display human-readable time format for app page (#1851)#2011
zhengchenyu merged 1 commit into
apache:branch-0.9from
maobaolong:cp1851

Conversation

@maobaolong

Copy link
Copy Markdown
Member

Display the human-readable time format in application page.

A long type timestamp really hard to get information from it.

Format the update time to human-readable.

Manual testing

What changes were proposed in this pull request?

(Please outline the changes and how this PR fixes the issue.)

Why are the changes needed?

(Please clarify why the changes are needed. For instance,

  1. If you propose a new API, clarify the use case for a new API.
  2. If you fix a bug, describe the bug.)

Fix: # (issue)

Does this PR introduce any user-facing change?

(Please list the user-facing changes introduced by your change, including

  1. Change in user-facing APIs.
  2. Addition or removal of property keys.)

No.

How was this patch tested?

(Please test your changes, and provide instructions on how to test it:

  1. If you add a feature or fix a bug, add a test to cover your changes.
  2. If you fix a flaky test, repeat it for many times to prove it works.)

…ormat for app page (apache#1851)

Display the human-readable time format in application page.

A long type timestamp really hard to get information from it.

Format the update time to human-readable.

Manual testing
@github-actions

github-actions Bot commented Aug 3, 2024

Copy link
Copy Markdown

@maobaolong maobaolong closed this Aug 3, 2024
@maobaolong maobaolong reopened this Aug 3, 2024
@zhengchenyu zhengchenyu merged commit 727ac17 into apache:branch-0.9 Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants