kamalcph commented on PR #14238: URL: https://github.com/apache/kafka/pull/14238#issuecomment-1686286807
> Do we need to implement dynamically configurable remote storage related things in a `DynamicRemoteLogManagerConfig` class similar to https://github.com/apache/kafka/pull/14243/files ? (I haven't looked into the code in detail yet so asking for your view point) I took a pass on #14243, there we are changing the broker level configs that are related to remote storage. We can extend the `DynamicRemoteLogManagerConfig` with dynamic remote log read/write quota, thread-pool size and so on. In this PR, we are changing the topic-level configs. `remote.storage.enable` config is applicable only at the topic level. -- 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
