[GitHub] [incubator-pinot] siddharthteotia opened a new pull request #4550: Support transform functions with AVG aggregation function

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

siddharthteotia opened a new pull request #4550:  Support transform functions with AVG aggregation function
URL: https://github.com/apache/incubator-pinot/pull/4550
 
 
   Support queries like:
   
   SELECT AVG(SUB(col1, col2) FROM foo
   SELECT AVG(SUB(DIV(col1, col2), DIV(col3, col4))) FROM foo
   
   They were failing in AvgAggregationFunction as TransformBlockValSet() did not implement getValueType() method
   
   Also added operand checks for transform functions for acceptable data types
   
   Testing:
   
   Added TransformQueriesTest to test both inner-segment and inter-segment, inter-server queries with custom data

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