FrankChen021 commented on PR #19875: URL: https://github.com/apache/druid/pull/19875#issuecomment-5186099850
@clintropolis Thanks for opening #19882. I reviewed the overlap and suggest keeping #19875 for its other extensions, while treating #19882 as the prerequisite for the Bloom filter portion: after #19882 is ready or merged, I will rebase #19875, keep #19882's BloomDimFilterTest/BaseFilterTest migration, and remove the temporary bridge and compatibility dependencies from #19875. #19876-#19878 have no file overlap with #19882 and should remain independent. I also found two CI issues in #19882 (File.mkdirs() forbidden-API usage and an inner class that should be static), which I am fixing there now. -- 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]
