Re: [DISCUSS] KIP-930: Tiered Storage Metrics

Hi Abhijeet, Thanks for this KIP, I pretty much agree with the renaming and new names look good to me. Cheers, Jorge. On Tue, 25 Jul 2023 at 12:56, Satish Duggana wrote: > Hi Abhijeet, > Thanks for keeping this KIP only to renaming the existing metrics for > better clarity. These new names

Re: [DISCUSS] KIP-930: Tiered Storage Metrics

Hi Abhijeet, Thanks for keeping this KIP only to renaming the existing metrics for better clarity. These new names look good to me. ~Satish. On Tue, 25 Jul 2023 at 13:12, Luke Chen wrote: > > Hi Abhijeet, > > Thanks for the KIP! > I don't have much preference for the name changing. > But if it

Re: [DISCUSS] KIP-930: Tiered Storage Metrics

Hi Abhijeet, Thanks for the KIP! I don't have much preference for the name changing. But if it could confuse other people, it's good to make it clear. Thank you. Luke On Tue, Jul 25, 2023 at 2:53 PM Abhijeet Kumar wrote: > Hi Kamal, > > As we discussed offline, I will rename this KIP so that

Re: [DISCUSS] KIP-930: Tiered Storage Metrics

Hi Kamal, As we discussed offline, I will rename this KIP so that it only captures the aspect of renaming the previously added metrics to remove ambiguity. I will create another KIP for RemoteIndexCache metrics and other relevant tiered storage metrics. On Tue, Jul 25, 2023 at 12:03 PM Kamal

Re: [DISCUSS] KIP-930: Tiered Storage Metrics

Hi Abhijeet, Thanks for the KIP! We are changing the metric names from what was proposed in the KIP-405 and adding new metrics for RemoteIndexCache. In the KIP, it's not clear whether we are renaming the aggregate broker level metrics for remote copy/fetch/failed-copy/failed-fetch. Are these

[DISCUSS] KIP-930: Tiered Storage Metrics

Hi All, I created KIP-930 for adding RemoteIndexCache stats and also to rename some tiered storage metrics added as part of KIP-405 to remove ambiguity.

[DISCUSS] KIP-930: Tiered Storage Metrics

Hi All, I created KIP-930 for adding RemoteIndexCache stats and also to rename some tiered storage metrics added as part of KIP-405 to remove ambiguity.