abhishekagarwal87 commented on issue #11929: URL: https://github.com/apache/druid/issues/11929#issuecomment-1101123251
Thank you for clarifying @gianm. @adarshsanjeev - FWIW, we already convert conditions on the `time` filter to intervals. The class that you want to look into is `MoveTimeFiltersToIntervals` and `Filtration`. I think that initially, we would want to support just time-based filters. We should try to extract the FULL filter specified in `DELETE WHERE` clause to intervals and if we can't do that, then throw an error. I think it's alright if we can't support all the filter combinations in the beginning. We just need to document what is supported and what is not. -- 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]
