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/03/28 20:43:12 UTC
[GitHub] [incubator-druid] gianm commented on issue #7370: Add SQL support
for time-ordered scans
gianm commented on issue #7370: Add SQL support for time-ordered scans
URL: https://github.com/apache/incubator-druid/issues/7370#issuecomment-477763460
> Scan is used if ordering by __time is specified
Scan is already used if ordering by `__time` is _not_ specified so I suppose this proposed change means that Select will _never_ be used by Druid SQL. That sounds ok to me, because in situations you describe (lots of segments or high thresholds), Select queries have resource usage issues anyway as described in #6088. Make sure to update the Druid SQL documentation to talk about how the queries are planned now.
👍
----------------------------------------------------------------
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