refactor: migrage Bitmap from arrow2 to arrow-rs - Githubissues

databendlabs / databend

𝗗𝗮𝘁𝗮, 𝗔𝗻𝗮𝗹𝘆𝘁𝗶𝗰𝘀 & 𝗔𝗜. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com

https://docs.databend.com

Other

7.71k stars 732 forks source link

refactor: migrage Bitmap from arrow2 to arrow-rs #16523

Open andylokandy opened 1 day ago

andylokandy commented 1 day ago

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

Tests

[x] Unit Test
[ ] Logic Test
[ ] Benchmark Test
[ ] No Test - Explain why

Type of change

[ ] Bug Fix (non-breaking change which fixes an issue)
[ ] New Feature (non-breaking change which adds functionality)
[ ] Breaking Change (fix or feature that could cause existing functionality not to work as expected)
[ ] Documentation Update
[x] Refactoring
[ ] Performance Improvement
[ ] Other (please describe):

This change is

Dousir9 commented 19 hours ago

It would be better to provide a rust benchmark to test the migrated functions in kernels(take.rs, concat.rs, ...) to ensure that there is no performance regression.