[GitHub] [incubator-pinot] siddharthteotia commented on issue #5807: Proposal: Deprecate PQL

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

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



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