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