Thanks @Ivan for reminding me. Ivan Kelly <iv...@apache.org> 于2021年9月7日周二 下午8:34写道:
> 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 > >> > > >> > > >