Skip to content

Improve rust log format#1084

Merged
richox merged 1 commit intoapache:masterfrom
cxzl25:log
Aug 4, 2025
Merged

Improve rust log format#1084
richox merged 1 commit intoapache:masterfrom
cxzl25:log

Conversation

@cxzl25
Copy link
Contributor

@cxzl25 cxzl25 commented Aug 1, 2025

Which issue does this PR close?

Closes #.

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Master

(+0.408s) [INFO] (stage: 0, partition: 0, tid: 0) - (partition=1) native execution finalizing
(+0.408s) [INFO] (stage: 0, partition: 0, tid: 0) - (partition=7) native execution finalizing
(+0.408s) [INFO] (stage: 0, partition: 0, tid: 0) - (partition=1) native execution finalized
(+0.408s) [INFO] (stage: 0, partition: 0, tid: 0) - (partition=7) native execution finalized

PR

2025-08-01 14:59:33.345 (+0.569s) [INFO] [blaze::rt:261] (stage: 0, partition: 0, tid: 0) - (partition=8) native execution finalizing
2025-08-01 14:59:33.345 (+0.569s) [INFO] [blaze::rt:269] (stage: 0, partition: 0, tid: 0) - (partition=8) native execution finalized
2025-08-01 14:59:33.345 (+0.570s) [INFO] [blaze::rt:269] (stage: 0, partition: 0, tid: 0) - (partition=9) native execution finalized

@richox
Copy link
Contributor

richox commented Aug 4, 2025

we had been using chrono to print formatted time in logging, however we noticed chrono failed to initialize in some of our yarn node of our production environment.
i would like to accept this change. one should use the old format if there are any production issues.

@richox richox merged commit d86aa33 into apache:master Aug 4, 2025
619 checks passed
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