showuon commented on PR #13828: URL: https://github.com/apache/kafka/pull/13828#issuecomment-1588961245
@divijvaidya @satishd , PR updated. Thanks. > 1. We probably want to update the KIP-405 here: https://cwiki.apache.org/confluence/display/KAFKA/KIP-405%3A+Kafka+Tiered+Storage#KIP405:KafkaTieredStorage-Configs and specify that this config is optional Updated. > 2. From the javadoc of RLMM > When this is configured all other > required properties can be passed as properties with prefix of 'remote.log.metadata.manager.listener > Can we please add a test to verify this? (asking because while constructing the rlmmProps, we don't pass any other configs with the listener prefix) I think we don't have this implemented. We should pass `remote.log.metadata.*` into RLMM based on KIP-405. Created [KAFKA-15083](https://issues.apache.org/jira/browse/KAFKA-15083) for this issue. Thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org