Hi Tom, The images seem to be stripped by the mailing list. Perhaps post them to the PR.
Cheers, Ivan On Tue, Sep 7, 2021 at 1:30 PM tom lee <tomlees...@gmail.com> wrote: > Thanks @Penghui for your comments. > > I added timezone in log pattern. Do other guys have any questions or > concerns about this change? Looking forward to your reply and discussion. > > before the change: > [image: before-change.jpg] > > after the change: > [image: after-change.jpg] > > Regards, > Tom > > PengHui Li <peng...@apache.org> 于2021年9月7日周二 下午4:56写道： > >> LGTM, I have added the tag `release/note-required` to make sure the change >> will be highlighted in the release note. >> >> Thanks, >> Penghui >> >> On Tue, Sep 7, 2021 at 4:41 PM tom lee <tomlees...@gmail.com> wrote: >> >> > Hello dev list, >> > >> > As mentioned in this issue(https://github.com/apache/pulsar/issues/8298 >> ), >> > there is no date in pulsar's log at present, only time. Sometimes it's >> not >> > easy to track problems. >> > >> > To facilitate problem tracking, unify the pulsar's format, we plan to >> > update pulsar's default log pattern. (HH:mm:ss.SSS -> yyyy-MM-dd >> > HH:mm:ss.SSS) >> > >> > However, considering that there may be many systems that parse logs and >> if >> > we change the pattern we are going to break things. So the community >> must >> > be informed and agree on this change. >> > >> > The PR is https://github.com/apache/pulsar/pull/11935. >> > >> > I posted this email here to attract more reviewers. >> > >> > Regards, >> > Tom >> > >> >
