You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/05/07 01:07:02 UTC

[GitHub] [incubator-druid] gianm commented on issue #7592: An question about groupby type SQL query

gianm commented on issue #7592: An question about groupby type SQL query
URL: https://github.com/apache/incubator-druid/issues/7592#issuecomment-489866800
 
 
   I can't find the issue number right now, but essentially the reason for this is that 'granularity' for groupBy does not work the way that you would expect for a SQL GROUP BY when there's an ORDER BY or a LIMIT clause, so it cannot be used by the SQL planner. I think the issue was that dimension ordering is applied within each granular bucket for groupBy, rather than across all buckets as you would expect with SQL.
   
   However, I would like to optimize the execution of `FLOOR(__time TO DAY)` enough that this is not an issue perf-wise. That seems like a good direction to me.

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