Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

increase log timestamp resolution to milliseconds #1342

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jelmd
Copy link
Contributor

@jelmd jelmd commented Dec 6, 2023

If --new-log-timestamp is enabled and the --new-log-timestamp-format ends with %N, the time gets formatted as UTC timestamp whereby %N gets replaced by the microseconds part of the current time.

@jelmd jelmd changed the title increase log timestamp resolution to microseconds increase log timestamp resolution to milliseconds Dec 6, 2023
@KangLin
Copy link
Contributor

KangLin commented Dec 7, 2023

It is recommended to use a mature logging library. #1344

@jelmd
Copy link
Contributor Author

jelmd commented Dec 7, 2023

Yes, but this requires a lot more work, refactoring. So 'til someone gets the time to do it, this fills the gap.

If --new-log-timestamp is enabled and the --new-log-timestamp-format
ends with %N, the time gets formatted as UTC timestamp whereby %N
gets replaced by the microseconds part of the current time.
@jelmd
Copy link
Contributor Author

jelmd commented Dec 8, 2023

fixes #1341

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.

None yet

3 participants