Repository navigation
Conversation
|
closing as i suspect its AI spam and doesnt fully address the root cause issue |
|
Per discussion on #11215, reopening this PR and marking it draft while @efegokdemir works through the feedback |
Signed-off-by: Efe Gökdemir <[email protected]>
efd6eb4 to
fa9d211
Compare
|
Addressed the non-zero case in Validation: |
Summary
MutableArrayData::try_extend_nulls(0)remains a no-op for empty unions. For non-empty sparse and dense unions, null extension now uses a nullable child field rather than assuming the first child is nullable.Fixes #11198
Testing
cargo test --locked --manifest-path arrow-data/Cargo.toml -p arrow-data --lib(64 passed)cargo fmt --all -- --checkgit diff --check HEAD^ HEADAI assistance was used during implementation.