siddharthteotia edited a comment on issue #5807: URL: https://github.com/apache/incubator-pinot/issues/5807#issuecomment-781713127
We can split this into 2 parts PQL endpoint removal from broker and controller. - The controller's query console endpoint (/pql) talks to broker's endpoint (/query). PQL cleanup from the engine - Remove Antlr parser - Remove PQL2Compiler - Don't convert PinotQueryToBrokerRequest: just pass PinotQuery inside BrokerRequest to server. Server can already handle it - Cleanup combine operator and broker reduce operators that have different code paths for PQL and SQL. - Cleanup tests to remove use of PQL compiler. The first task of endpoint removal can be done by end of Feb. This will at least ensure no one can use PQL to query Pinot even though PQL exists in the engine. Cleanup of PQL from the engine can be a subsequent task. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
