chia7712 commented on PR #15775:
URL: https://github.com/apache/kafka/pull/15775#issuecomment-2075158081

   > If we start to move related kraft classes to raft module like 
KafkaRaftManager this will be tricky as now 
org.apache.kafka.raft.KafkaRaftManager needs 
org.apache.kafka.server.config.KafkaConfig, 
org.apache.kafka.server.config.KafkaConfig needs 
org.apache.kafka.raft.RaftConfig and org.apache.kafka.server.BrokerServer needs 
org.apache.kafka.raft.KafkaRaftManager
   
   `org.apache.kafka.server.config.KafkaConfig` -> this means `KafkaConfig` 
will be moved to server module in the future? If so, is it weird to make raft 
module depend on server module? Please correct me if I misunderstand anything.


-- 
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

Reply via email to