Hi Jian, +1 (binding). Thanks for the KIP!
Thanks, Kamal. On Wed, Dec 10, 2025 at 10:37 AM jian fu <[email protected]> wrote: > sorry. resent with formatted email content > > Hi all, I'd like to initiate the vote for KIP-1235 right away as it's a > minor change with minimal risk. > > Summary: Currently, __remote_log_metadata relies on the broker default for > min.isr (typically 1). With min.isr=1 and factor=3, acknowledged writes can > be lost if the leader crashes before replication completes, making remote > data permanently unreachable. > > What's more, the similar topic: __transaction_state sets min.isr=2 via > transaction.state.log.min.isr.We should provide the same durability for > __remote_log_metadata. Proposed Change: Add one new broker config: > remote.log.metadata.topic.min.isr (default: 2) When creating > __remote_log_metadata, TopicBasedRemoteLogMetadataManager will set > min.insync.replicas=2 at the topic level. > > KIP WIKI: > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-1235%3A+Correct+the+default+min.insync.replicas+to+2+for+the+__remote_log_metadata+topic > > Discussion Thread: > https://lists.apache.org/thread/016mjw3rrcqnt8475bxr3b3npm0t32xl > PR: https://github.com/apache/kafka/pull/20811 > > Looking forward to your feedback! > Thanks! > > Regards > Jian > > jian fu <[email protected]> 于2025年12月10日周三 12:51写道： > > > Hi all, I'd like to initiate the vote for KIP-1235 right away as it's a > > minor change with minimal risk. Summary: Currently, __remote_log_metadata > > relies on the broker default for min.isr (typically 1). With min.isr=1 > and > > factor=3, acknowledged writes can be lost if the leader crashes before > > replication completes, making remote data permanently unreachable. What's > > more, the similar topic: __transaction_state sets min.isr=2 via > > transaction.state.log.min.isr.We should provide the same durability for > > __remote_log_metadata. Proposed Change: Add one new broker config: > > remote.log.metadata.topic.min.isr (default: 2) When creating > > __remote_log_metadata, TopicBasedRemoteLogMetadataManager will set > > min.insync.replicas=2 at the topic level. KIP WIKI: > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-1235%3A+Correct+the+default+min.insync.replicas+to+2+for+the+__remote_log_metadata+topic > > Discussion Thread: > > https://lists.apache.org/thread/016mjw3rrcqnt8475bxr3b3npm0t32xl PR: > > https://github.com/apache/kafka/pull/20811 Looking forward to your > > feedback! Thanks! Regards Jian > > >
