[GitHub] b-slim commented on issue #6583: support rollup mode [COUNT() -> SUM(count)]

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

b-slim commented on issue #6583: support rollup mode [COUNT() -> SUM(count)]
URL: https://github.com/apache/incubator-druid/issues/6583#issuecomment-437126236
 
 
   I don't call this 
   
   > A user runs "SELECT count(field)" and gets a result of 1000
   >The user then runs "SELECT field" but only gets back 10 rows.
   
   a detail, it is actually correctness, am assuming your users are actually SQL fanatics thus they care about correctness, FYI that might be also the same issue with Average, once the data is rolled up you need to break SQL AVG into SUM and Sum(SUM(__count))  as well not sure if the current SQL integration is doing that.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org