OmniaGM commented on PR #16665: URL: https://github.com/apache/kafka/pull/16665#issuecomment-2298443356
> @OmniaGM Both `TransactionStateManagerConfig` and `TransactionLogConfig` are small file, and they are normally called together. Hence, do you consider merging them into one file? They are called together only `KafkaConfig`, and `TransactionStateManager`. While `TransactionLogConfig` is called in `DynamicBrokerConfig`. I split them into 2 previously because the comment on them in KafkaConfig back then suggested they are different domains. I guess we can move `TransactionLogConfig` into `TransactionStateManagerConfig` -- 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
