apache / arrow-rs

Official Rust implementation of Apache Arrow
https://arrow.apache.org/
Apache License 2.0
2.62k stars 802 forks source link

Fix Dictionary logical nulls for RunArray/UnionArray Values #6740

Closed findepi closed 3 days ago

findepi commented 5 days ago

Which issue does this PR close?

None

Rationale for this change

For DictionaryArray, methods logical_nulls and logical_null_count would return incorrect result if the underlying values had different physical and logical nullability.

What changes are included in this PR?

Fix DictionaryArray.logical_nulls

Are there any user-facing changes?

yes