Hi Pinot Devs,
This is Jia from Pinot team @ LinkedIn. Currently @Siddharth
Teotia<mailto:[email protected]>, @Dino
Occhialini<mailto:[email protected]>, and I are integrating a config
recommendation engine within LI. For cleaner interface and better user
experience, it is good to have a rest endpoint on controller; and therefore the
engine should also resides in controller. To achieve this, we are proposing to
move its dependencies, request optimizer and parser factory, from broker to
core. We have explored other options as -- (1) moving engine to broker, but
logically having this API on broker does not make sense and will be hard to
invoke, (2) moving dependencies to common, will require more dependencies (e.g.
part of query context) to be moved from core to common to avoid cycle, which is
not clean; and so finally moving optimizer to core is a viable solution.
Please feel free to give us feedback! Thanks!
