Hi Chia, Thanks for the review!
> the `PartitionSize` does not include the `RemoteLogSize`, right? Yes, it is the local-log size. > Should we add similar fields for `TotalBytes` and `UsableBytes`? TotalBytes and UsableBytes may not be applicable for remote storage. As the storage implementer can add multiple buckets for a Kafka cluster. Thanks, Kamal, On Mon, Jul 21, 2025 at 10:39 AM Chia-Ping Tsai <chia7...@gmail.com> wrote: > hi PoAn > > chia_00: > > should `setIncludeRemoteInfo(boolean)` be renamed to ` > includeRemoteInfo(boolean)`? > > chia_01: > > the `PartitionSize` does not include the `RemoteLogSize`, right? > > chia_02: > > Should we add similar fields for `TotalBytes` and `UsableBytes`? > > Best, > Chia-Ping > > ally heev <allyh...@gmail.com> 於 2025年7月17日 週四 下午11:38寫道： > > > On 2025/06/16 14:32:34 PoAn Yang wrote: > > > Hi all, > > > > > > I would like to start a discussion thread about KIP-1187. > > > > > > Please take a look and feel free to share any thought. > > > > > > https://cwiki.apache.org/confluence/x/sYkhFg > > > > > > Thanks, > > > PoAn > > > > Minor comment: > > *describeLogDirs *literally means describing the log dirs on the broker > > right? Maybe we want to refactor around it or create new tools/functions > > for describing remoteLog > > >
