Hi All: Update one tiny change:
According to Chia-Ping Tsai and Kamal’s good suggestion on code review, we narrowed the interface scope to avoid over-engineering at this phase. Detail change: Moved BrokerReadyCallback from package org.apache.kafka.server.common to org.apache.kafka.server.log.remote.metadata.storage. We can move back later if another module requires it in future. Regards Jian (jiafu1115) jian fu <[email protected]> 于2025年10月24日周五 18:51写道： > > Hi all, > > Thanks for voting on this KIP. The KIP is approved. > Kamal Chandraprakash +1 (binding) > Andrew Schofield +1 (binding) > Luke +1 (binding) > Anatolii Popov +1 (non-binding) > Jhen-Yung Hsu +1 (non-binding) Thanks, > Jian > > jian fu <[email protected]> 于2025年10月23日周四 16:05写道： > >> Hi Chia-Ping： >> >> Thanks for your comments. >> >> Hi all voters: >> >> Let me update one message: >> I updated the KIP part content and related code according to Chia-Ping's >> suggestions. Let me paste the change here: >> >> having TopicBasedRemoteLogMetadataManager (instead of >> RemoteLogMetadataManager) implement BrokerReadyCallback. It can also solve >> Kamal Chandraprakash‘s concern. >> >> public interface RemoteLogMetadataManager extends BrokerReadyCallback , >> Configurable, Closeable >> => >> public class TopicBasedRemoteLogMetadataManager implements >> BrokerReadyCallback, RemoteLogMetadataManager >> >> Regards >> Jian >> >> Chia-Ping Tsai <[email protected]> 于2025年10月23日周四 09:14写道： >> >>> hi all voters, >>> >>> I have an alternative idea [0] for addressing the issue without exposing >>> a new interface as a public Kafka API. >>> >>> While I'm fine with the new Kafka API since it has received three >>> binding votes, my preference remains to fix the issue with the most minimal >>> changes possible >>> >>> [0] https://lists.apache.org/thread/0gkrb44rb6tj4ocn4qhy63rt64tsw9tl >>> >>> Best, >>> Chia-Ping >>> >>> On 2025/10/21 13:52:24 Andrew Schofield wrote: >>> > Hi, >>> > When I first reviewed this KIP, I was a bit unconvinced about the need >>> but >>> > the generic broker ready interface looks good to me. >>> > >>> > +1 (binding) >>> > >>> > Thanks, >>> > Andrew >>> > >>> > >>> > > On 21 Oct 2025, at 14:39, jian fu <[email protected]> wrote: >>> > > >>> > > Hi Luke: >>> > > >>> > > Thanks for your review ！ >>> > > >>> > > Hi Kafka committers: >>> > > >>> > > As of now, we’re just one binding vote short to move forward. We’ve >>> been in >>> > > the voting phase for three months.— if anyone has a moment, could you >>> > > please help take a look at this KIP? Thanks a lot! >>> > > Anatolii Popov +1 (non-binding) >>> > > Jhen-Yung Hsu +1 (non-binding) >>> > > Kamal Chandraprakash +1 (binding) >>> > > Luke +1 (binding) >>> > > >>> > > Luke Chen <[email protected]> 于2025年10月20日周一 19:22写道： >>> > > >>> > >> +1 (binding) from me. >>> > >> >>> > >> Thanks. >>> > >> Luke >>> > >> >>> > >> On Mon, Oct 13, 2025 at 5:24 PM jian fu <[email protected]> >>> wrote: >>> > >> >>> > >>> Hi Kafka committers: >>> > >>> >>> > >>> Bumping the thread once more. Help to take a look and vote. We >>> still >>> > >> need 2 >>> > >>> binding votes ！ BTW: This KIP isn’t a complex one. >>> > >>> >>> > >>> jian fu <[email protected]> 于2025年10月6日周一 22:12写道： >>> > >>> >>> > >>>> Hi Kafka commiters: >>> > >>>> >>> > >>>> I’ve summarized the current voting status as follows. We still >>> need two >>> > >>>> binding votes to move forward. Could anyone take a look at the >>> KIP? >>> > >>> Thanks. >>> > >>>> >>> > >>>> Anatolii Popov KIP! +1 (non-binding) >>> > >>>> Kamal Chandraprakash KIP! +1 (binding) >>> > >>>> Jhen-Yung Hsu KIP! +1 (non-binding) >>> > >>>> >>> > >>>> The KIP link is: >>> > >>>> >>> > >>> >>> > >> >>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-1197%3A+Introduce+new+method+to+improve+the+TopicBasedRemoteLogMetadataManager%27s+initialization >>> > >>>> >>> > >>>> Anatolii Popov <[email protected]> 于2025年10月3日周五 >>> > >> 20:10写道： >>> > >>>> >>> > >>>>> Hi Jian, >>> > >>>>> >>> > >>>>> Thanks for the KIP! +1 (non-binding) >>> > >>>>> >>> > >>>>> Regards, >>> > >>>>> Anatolii. >>> > >>>>> >>> > >>>>> On Fri, Oct 3, 2025 at 2:34 AM jian fu <[email protected]> >>> wrote: >>> > >>>>> >>> > >>>>>> Still need 2 votes (binding) >>> > >>>>>> >>> > >>>>>> BTW. I don't know why this vote (no binding) is not shown in >>> this >>> > >>>>>> message thread . So paste the mail content here for tracking. >>> > >>>>>> >>> https://lists.apache.org/thread/w0zn73qqhgo41hzxshq8r1okzhsh0cbw >>> > >>>>>> >>> > >>>>>> [image: image.png] >>> > >>>>>> >>> > >>>>>> jian fu <[email protected]> 于2025年8月21日周四 15:47写道： >>> > >>>>>> >>> > >>>>>>> Hi All, >>> > >>>>>>> Could anyone else kindly take a look at this KIP? >>> > >>>>>>> It’s not a complex one — we just need to add a default >>> interface >>> > >>> method >>> > >>>>>>> in order to move the PR forward. >>> > >>>>>>> Many thanks! >>> > >>>>>>> >>> > >>>>>> >>> > >>>>>> >>> > >>>>>> -- >>> > >>>>>> Regards >>> > >>>>>> >>> > >>>>>> Fu.Jian >>> > >>>>>> >>> > >>>>>> >>> > >>>>>> >>> > >>>>> >>> > >>>> >>> > >>>> >>> > >>>> -- >>> > >>>> Regards >>> > >>>> >>> > >>>> Fu.Jian >>> > >>>> >>> > >>>> >>> > >>>> >>> > >>> >>> > >>> -- >>> > >>> Regards >>> > >>> >>> > >>> Fu.Jian >>> > >>> >>> > >> >>> > >>> > >>> >> >> >> >> >> > >
