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