[Enhancement] The bitmap_hash function can be implemented using murmur_hash3_128

apache / doris

Apache Doris is an easy-to-use, high performance and unified analytics database.

https://doris.apache.org

Apache License 2.0

12.66k stars 3.27k forks source link

[Enhancement] The bitmap_hash function can be implemented using murmur_hash3_128 #8435

Open syb853553110 opened 2 years ago

syb853553110 commented 2 years ago

Search before asking

[X] I had searched in the issues and found no similar issues.

Description

The bitmap_hash function can be implemented using murmur_hash3_128

Solution

The bitmap_hash function can be implemented using murmur_hash3_128, which is more efficient than murmur_hash3_32

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

zbtzbtzbt commented 2 years ago

I think this modification will be incompatible with old data @syb853553110 https://doris.apache.org/zh-CN/sql-reference/sql-functions/bitmap-functions/bitmap_hash.html#description

syb853553110 commented 2 years ago

I think this modification will be incompatible with old data @syb853553110 https://doris.apache.org/zh-CN/sql-reference/sql-functions/bitmap-functions/bitmap_hash.html#description

Is it possible to add a method? For example bitmap_hash128