yashmayya opened a new pull request, #14071:
URL: https://github.com/apache/pinot/pull/14071

   - Null handling support for aggregation functions was added in 
https://github.com/apache/pinot/pull/9086, 
https://github.com/apache/pinot/pull/12227, 
https://github.com/apache/pinot/pull/12271, 
https://github.com/apache/pinot/pull/12252, 
https://github.com/apache/pinot/pull/12235 etc.
   - However, most of these only added support to aggregations without group by 
and aggregations grouped by SV columns. This means that if null handling for 
queries is enabled and an aggregation with a group by clause on an MV column is 
executed, it will either lead to incorrect results or an error (if the 
aggregation result holder for the function differs between the null handling 
modes).
   - This patch adds null handling support for aggregations grouped by MV 
columns for all the aggregation functions that already support null handling.


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to