+1(binding)
Best, Mattison On 20 Jun 2023 at 10:45 +0800, PengHui Li <peng...@apache.org>, wrote: > +1 (binding) > > Thanks, > Penghui > > On Tue, Jun 20, 2023 at 10:40 AM Yubiao Feng > <yubiao.f...@streamnative.io.invalid> wrote: > > > Voting +1 (non-binding) > > > > Thanks > > Yubiao Feng > > > > On Mon, Jun 19, 2023 at 5:21 PM Asaf Mesika <asaf.mes...@gmail.com> wrote: > > > > > > Voting +1 (non-binding) > > > > > > > > On Fri, Jun 16, 2023 at 12:23 PM guo jiwei <techno...@apache.org> wrote: > > > > > > > > > > @Asaf Thanks, I have addressed the comment. > > > > > > > > > > > > Regards > > > > > > Jiwei Guo (Tboy) > > > > > > > > > > > > > > > > > > On Fri, Jun 16, 2023 at 3:55 AM Asaf Mesika <asaf.mes...@gmail.com> > > > > wrote: > > > > > > > > > > > > > > -1 (non-binding) > > > > > > > > > > > > > > > > I'm perfectly ok with the idea; just please fix the document. It > > looks > > > > > > too > > > > > > > > messy. Even 1 paragraph changes can look neat and clean. > > > > > > > > I left notes in the draft PR you opened for the pip. > > > > > > > > > > > > > > > > I'll change my non-binding vote once that's done. > > > > > > > > > > > > > > > > On Thu, Jun 15, 2023 at 11:07 AM guo jiwei > > > > > > > > <techno...@apache.org> > > > > wrote: > > > > > > > > > > > > > > > > > > Hi, community: > > > > > > > > > > The metrics are all started with `pulsar_`, so that both > > > > > > > > > > users > > > > and > > > > > > > > > > operators can quickly find the metrics of the entire system > > > > > > > > > > through > > > > > > > > > > this prefix. However, due to some other reasons, it was > > > > > > > > > > found that > > > > > > > > > > `topic_load_times` was missing the prefix, so want to get > > > > > > > > > > it right. > > > > > > > > > > In the master branch : > > > > > > > > > > * `pulsar_topic_load_times`: Add this new metric which has > > > > > > > > > > the > > > > > > same > > > > > > > > > > meaning as `topic_load_times` > > > > > > > > > > * `topic_load_times`: Mark this metric as deprecated and > > remove > > > > > > it > > > > > > > > in > > > > > > > > > > the next version > > > > > > > > > > > > > > > > > > > > PIP: https://github.com/apache/pulsar/pull/20518 > > > > > > > > > > > > > > > > > > > > Regards > > > > > > > > > > Jiwei Guo (Tboy) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
