[GitHub] gaodayue opened a new pull request #6246: add a sql option to force user to specify time condition

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

gaodayue opened a new pull request #6246: add a sql option to force user to specify time condition
URL: https://github.com/apache/incubator-druid/pull/6246
 
 
   One downside of SQL's flexibility is that users tend to forget adding time filter when writing queries, which is usually very bad for timeseries data analysis and causes stability issue.
   
   Since native query requires users to specify "intervals", I think it's reasonable to force SQL users to specify __time filter in many cases.
   
   This PR adds a new option `druid.sql.planner.forceTimeCondition` to support such needs. It defaults to false though so that the default behaviour is the same as before.

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