Jaehui-Lee commented on PR #7653: URL: https://github.com/apache/hbase/pull/7653#issuecomment-3834981272
@Apache9 Thank you for the review! I've addressed your feedback: - https://github.com/apache/hbase/pull/7653/commits/1e6152e895ffaaa391f5a5696126b1edd6c16485 - https://github.com/apache/hbase/pull/7653/commits/237dce26cd02fc1b6799c1cce963bda65accc163 - The global configuration (`io.storefile.bloom.filter.impl`) was effectively dead code, and column family level configuration should be sufficient for now, so I removed it. > So the ribbon filter implementation in this PR is implemented from scratch? Yes, this is implemented from scratch based on: 1. The original Ribbon Filter paper: https://arxiv.org/abs/2103.02515 2. The reference implementation in https://github.com/FastFilter/fastfilter_cpp by the paper authors -- 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]
