adoroszlai commented on PR #7338:
URL: https://github.com/apache/ozone/pull/7338#issuecomment-2440989220
> Sure,my current time with time zone is 2024-10-26T15:09:49
>
> Before modification:
>
> > Container Summary Report generated at 2024-10-26T07:09:49Z
> > ==========================================================
>
> After modification:
>
> > Container Summary Report generated at 2024-10-26T15:09:49
> > ==========================================================
It seems to me the change actually removes timezone information from the
output. `07:09:49Z` may be inconvenient, but it's clear that time is UTC.
`15:09:49` could be different time in various timezones.
So we may need to consider including timezone in the output.
Also:
Is output of other Ozone commands consistent in how they display time?
(date/time format, display timezone (UTC vs. local))
- If so, is this change making it consistent with other commands?
- If not:
- What kind of different output formats do we have?
- Is this the only command that needs to be changed?
- Is the new format (local time + time zone) the desired output format we
should standardise on?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
