Jaehui-Lee commented on PR #7653: URL: https://github.com/apache/hbase/pull/7653#issuecomment-3797786156
> For me I prefer we introduce a new configuration which indicates the bloom filter implementation? The current BloomType just indicates that how we want to apply the bloom filter, not the implementation of the bloom filter itself. @Apache9 Thank you for the review! I've added a new `BloomFilterImpl` enum (BLOOM/RIBBON) separate from `BloomType`, with per-table (`BLOOMFILTER_IMPL`) and global (`io.storefile.bloom.filter.impl`) configuration support. -- 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]
