[GitHub] [incubator-pinot] siddharthteotia opened a new pull request #4648: Enable DISTINCT and throw exceptions for not-supported cases

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

siddharthteotia opened a new pull request #4648: Enable DISTINCT and throw exceptions for not-supported cases
URL: https://github.com/apache/incubator-pinot/pull/4648
 
 
   (1) Throw parse exception (after building AST) if DISTINCT query
   has ORDER BY. Done for both PQL compilation and Calcite SQL
   compilation paths
   
   (2) Throw runtime exception if DISTINCT query has one or more
   multi-value columns.
   
   Added unit tests for both cases. When the support is implemented,
   we can remove the tests and exceptions.
   
   Also, don't take star-tree path in AggregationPlanNode for DISTINCT
   query.
   
   cc @Jackie-Jiang , @mayankshriv 

----------------------------------------------------------------
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


With regards,
Apache Git Services

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