clolov commented on PR #14161: URL: https://github.com/apache/kafka/pull/14161#issuecomment-1697616881
> Looks good to me. One question I have is regarding dynamically changing broker properties to disable TS. Is that possible? Is yes, then we need to add more tests here. > > (will approve & merge once you can clarify this question please) No, as far as I have checked the code + manually tested none of the configuration in https://github.com/apache/kafka/blob/0912ca27e2a229d2ebe02f4d1dabc40ed5fab0bb/storage/src/main/java/org/apache/kafka/server/log/remote/storage/RemoteLogManagerConfig.java#L37 are dynamic -- 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
