[GitHub] QiuMM edited a comment on issue #5869: Period load rules should include the future by default

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

QiuMM edited a comment on issue #5869: Period load rules should include the future by default
URL: https://github.com/apache/incubator-druid/issues/5869#issuecomment-425308007
 
 
   @jihoonson currently, we can edit a `PeriodDropRule` like this:
   ```json
   {
     "type" : "dropByPeriod",
     "period" : "P1D"
   }
   ```
   The period is from some time in the past to the current time, so this example will drop last one day data. I think `PeriodDropRule` is a impractical rule for it will always drop recent data. So people who want to retain the last 30 days data, they can not use this `PeriodDropRule` but have to do like this: load 30 days, drop forever. Then because of `drop forever` cause issues that @gianm have listed above.
   
   Maybe we can remove the `PeriodDropRule` because I think few people would like to use it. Then add a new drop rule which will drop data older than a configured date. I also want a rule like this.

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