Thanks @Michael for your comments and suggestions. I just sent it to the users ML.
Michael Marshall <mikemars...@gmail.com> 于2021年9月8日周三 上午5:15写道： > Thanks for posting this on the dev mailing list, Tom. Would you also post > it to the users ML (us...@pulsar.apache.org)? > > Thanks, > Michael > > On Tue, Sep 7, 2021 at 7:36 AM tom lee <tomlees...@gmail.com> wrote: > > > 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 > > > >> > > > > >> > > > > > > > > > >
