[GitHub] [incubator-druid] gianm commented on issue #7793: Enable SQL by default

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

gianm commented on issue #7793: Enable SQL by default
URL: https://github.com/apache/incubator-druid/issues/7793#issuecomment-497177980
 
 
   I think the only real downside is that there will be slightly more load on historicals (due to extra metadata queries) and memory use on the broker will be somewhat higher (enabling SQL activates a metadata cache). But these effects should be small and I think they are worth it, since SQL is rad. So I think enabling it by default is good. By the way, most of our customers (at Imply) have SQL enabled.
   
   SQL can only do things that native queries could also do (they simply translate into native queries) so I don't think cluster operators getting unexpected SQL queries would be a serious issue.

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