Skip to content

Conversation

@maobaolong
Copy link
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
Copy link

github-actions bot commented Aug 3, 2024

Test Results

 2 370 files  ±0   2 370 suites  ±0   4h 40m 42s ⏱️ +17s
   918 tests ±0     917 ✅ ±0   1 💤 ±0  0 ❌ ±0 
10 645 runs  ±0  10 631 ✅ ±0  14 💤 ±0  0 ❌ ±0 

Results for commit 5933088. ± Comparison against base commit 9ab8b34.

♻️ This comment has been updated with latest results.

@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