lingguang commented on issue #12434: URL: https://github.com/apache/druid/issues/12434#issuecomment-1731601771
No, that's not a violation of the bucket size. Instead it is why it is called P7D not PT168H (7*24H). If you need keep the same time duration you should use PT168H. The P (period) part (Y/M/D/W) should be based on the calendar. The lib org.joda.time.chrono.ISOChronology which used in PeriodGranularity class follows the ISO8601 standard and works as expected. So the code here just has a minor defect, -14D-7D introduced problem because of the time gap of DST transition, -21D should be OK. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
