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
> >> >
> >>
> >
>

Reply via email to