cmccabe commented on pull request #10787:
URL: https://github.com/apache/kafka/pull/10787#issuecomment-853335116


   > I would personally not move these classes unless we believe we will use 
them from other modules. They're a bit specialized, especially the timeline 
ones.
   
   `KafkaEventQueue` is already used in `core`. It's just an event queue; there 
is nothing metadata-specific about it, so I think it belongs in 
`server-common`. The timeline data structures are not used anywhere else, 
however.
   
   I will revise the PR to move just the queue and not the timeline data 
structures.


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to