Hi Jian, Thanks for the KIP!
I went through the proposed solution and the discussions on GitHub and I wonder if you considered a more generic solution for notifying not only RLMM but other potential listeners as well? I think there are several places that could benefit from this and also it could improve testability quite a bit. Regards, Anatolii. On Mon, Aug 18, 2025 at 11:56 AM jian fu <fujian1...@gmail.com> wrote: > Hi Kamal > Thanks for your comments! > Regards > Jian > > Kamal Chandraprakash <kamal.chandraprak...@gmail.com> 于2025年8月12日周二 > 18:36写道： > > > Hi Jian, > > > > Thanks for the KIP! > > > > The newly introduced onBrokerReadyForRequests API in the > > RemoteLogMetadataManager (RLMM) > > is primarily needed for the topic-based RLMM (TBRLMM) implementation. > This > > API may not be necessary > > for other implementations that do not rely on Kafka to store remote log > > metadata. Given that TBRLMM is > > packaged as the default (out-of-the-box) implementation, this design > > approach seems reasonable to me. > > > > Thanks, > > Kamal > > > > On Fri, Jul 25, 2025 at 11:58 AM jian fu <fujian1...@gmail.com> wrote: > > > > > Hi Everyone: > > > Nice to meet you. > > > > > > I created one KIP to request your review. > > > KIP-1197: Introduce new method to improve the > > > TopicBasedRemoteLogMetadataManager's initialization > > > < > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-1197%3A+Introduce+new+method+to+improve+the+TopicBasedRemoteLogMetadataManager%27s+initialization > > > > > > > > > > The PR: > > > https://github.com/apache/kafka/pull/20203/files > > > > > > Thanks. > > > > > > > > > Regards Fu.Jian > > > > > > > > -- > -- Anatolii Popov Senior Software Developer, *Aiven OY* m: +358505126242 w: aiven.io e: anatolii.po...@aiven.io <https://www.facebook.com/aivencloud> <https://www.linkedin.com/company/aiven/> <https://twitter.com/aiven_io>
