albertobastos commented on PR #15502: URL: https://github.com/apache/pinot/pull/15502#issuecomment-2827115779
@Jackie-Jiang I see, my bad. Reversed it but at least ensuring a `resultTable.numRows` is properly set within `getEmptyBrokerOnlyResponse`. Don't think that's enough, though. For cases when the query is asking for total aggregations (such as `select avg(x), sum(y), count(z) from t`, makes no sense to return no rows and is legit by the user to expect that at least one row with zero values will be returned. I wonder if we could detect this particular case (projection with only aggregated functions) and fill a zero response then. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
