[Kernel] Fix RoaringBitmapArray create/add methods. Closes issue #3881 - Githubissues

delta-io / delta

An open-source storage framework that enables building a Lakehouse architecture with compute engines including Spark, PrestoDB, Flink, Trino, and Hive and APIs

https://delta.io

Apache License 2.0

7.62k stars 1.71k forks source link

[Kernel] Fix RoaringBitmapArray create/add methods. Closes issue #3881 #3882

Open YotillaAntoni opened 6 days ago

YotillaAntoni commented 6 days ago

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[X] Kernel
[ ] Other (fill in here)

Description

Resolves #3881

The bitmaps array should be initialized in the create path.
The expandBitMaps method should set new bitmaps from the old length up to the new length, instead of overwriting the old ones.

Also adds a toArray method mimicking the one provided by the scala version which the class is based on.

How was this patch tested?

Added unit tests.

Does this PR introduce any user-facing changes?

No