[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #5692: Decouple pinot-tools dependency on pinot-server, pinot-controller and pinot-broker

[GitBox <gi...@apache.org>]

Jackie-Jiang commented on issue #5692:
URL: https://github.com/apache/incubator-pinot/issues/5692#issuecomment-657747108


   Thanks for the suggestion.
   It is not possible to remove pinot-controller/broker/server dependency from pinot-tools because the tools package need to support starting controller/broker/server instance. But it is a great idea to add a separate client api module for admin purpose and have minimal dependencies.
   We are planning to re-organize `pinot-common` and `pinot-core` and split them into api/spi modules and implementation modules. Will add another client api module when we design the new package hierarchy.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org