yashmayya commented on PR #17058: URL: https://github.com/apache/pinot/pull/17058#issuecomment-3439285481
@Jackie-Jiang thoughts on the backward compatibility issue here? Are we okay with changing the return type for `MIN(longCol)` / `MAX(longCol)` from `DOUBLE` to `LONG` in SSE? It's already `LONG` for MSE (the aggregation result is currently cast form double to long). Also what about `SUM(longCol)`? Right now, we return `DOUBLE` in SSE which might lead to precision loss, but changing it to `LONG` here will mean a tradeoff for potential overflow. Again, in MSE today we cast the returned double value to long. -- 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]
