dang-stripe commented on PR #18471: URL: https://github.com/apache/pinot/pull/18471#issuecomment-4494721306
@yashmayya @Jackie-Jiang I updated the PR to only address the null behavior in count agg for now. To align with postgres behavior, I was thinking the aggregation interface has a `getMseEmptyInputResult` that defaults to null, but is overridden by all the count family aggs to return 0. This way each new aggregation type will handle null correctly by default. This way each agg function doesn't need to handle null specifically. -- 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]
