Re: [DISCUSS] Update the default log pattern

2021-09-07 Thread tom lee
Thanks @Michael for your comments and suggestions. I just sent it to the users ML. Michael Marshall 于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

Re: [DISCUSS] Update the default log pattern

2021-09-07 Thread Michael Marshall
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 wrote: > Thanks @Ivan for reminding me. > > Ivan Kelly 于2021年9月7日周二 下午8:34写道: > > > Hi Tom, > > > > The images seem to

Re: [DISCUSS] Update the default log pattern

2021-09-07 Thread tom lee
Thanks @Ivan for reminding me. Ivan Kelly 于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 wrote: > > > Thanks @Penghui for your comments. > > > > I added timezon

Re: [DISCUSS] Update the default log pattern

2021-09-07 Thread Ivan Kelly
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 wrote: > Thanks @Penghui for your comments. > > I added timezone in log pattern. Do other guys have any questions or > concerns about this change? Lookin

Re: [DISCUSS] Update the default log pattern

2021-09-07 Thread tom lee
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 于

Re: [DISCUSS] Update the default log pattern

2021-09-07 Thread PengHui Li
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 wrote: > Hello dev list, > > As mentioned in this issue(https://github.com/apache/pulsar/issues/8298), > there is no date in