fix(arrow/array): honor field nullability in comparison and JSON marshaling - #918

Closed

zeroshade wants to merge 18 commits into

apache:mainfrom

zeroshade:nullability-fixes

Commits on Jul 9, 2026

Separate public API from impl of comparison functions

serramatutu
authored and
zeroshade
committed
36599a8
Add `nullable` to `equalOption`

serramatutu
authored and
zeroshade
committed
29b75f0
Check for nullable opt in most array compare implementations

serramatutu
authored and
zeroshade
committed
fbfe0ca
Add stricter tests to null JSON in record and struct

serramatutu
authored and
zeroshade
committed
1da53ae
`AppendEmptyValue()` if field is nullable

serramatutu
authored and
zeroshade
committed
7dc69ee
Add `nullable` argument to `GetOneForMarshal()`

serramatutu
authored and
zeroshade
committed
554eddc
Make struct and record use `field.Nullable` when serializing

serramatutu
authored and
zeroshade
committed
6fb4438
Fix tests that were implicitly depending on wrong nullable semantics

serramatutu
authored and
zeroshade
committed
ad78a63
Post-rebase fixes for CI: JSON int precision, GetOneForMarshal, TinyGo
zeroshade
committed
0cbfe2a
Make nested nullability field-local in comparison and union marshal
zeroshade
committed
719f973
Make exact Equal and chunked comparison field-local too
zeroshade
committed
98c1bbe
Preserve nullability option in exact chunked comparison
zeroshade
committed
c084d40
Address review feedback on nullability-aware comparison
zeroshade
committed
39d9f56
Compare record/table fields ignoring metadata
zeroshade
committed
13e84dd
Address roborev findings on record/table field comparison
zeroshade
committed
4590470
Avoid breaking arrow.Array with optional NullableMarshaler
zeroshade
committed
94644b3
Thread field-local nullability through list, union, and REE
zeroshade
committed
a474f1f
Make run-end-encoded value comparison symmetric in ValueNullable
zeroshade
committed
45fef5e

Navigation Menu

Uh oh!

fix(arrow/array): honor field nullability in comparison and JSON marshaling - #918

fix(arrow/array): honor field nullability in comparison and JSON marshaling#918
zeroshade wants to merge 18 commits into
apache:mainfrom
zeroshade:nullability-fixes

Commits on Jul 9, 2026

Separate public API from impl of comparison functions

Add `nullable` to `equalOption`

Check for nullable opt in most array compare implementations

Add stricter tests to null JSON in record and struct

`AppendEmptyValue()` if field is nullable

Add `nullable` argument to `GetOneForMarshal()`

Make struct and record use `field.Nullable` when serializing

Fix tests that were implicitly depending on wrong nullable semantics

Post-rebase fixes for CI: JSON int precision, GetOneForMarshal, TinyGo

Make nested nullability field-local in comparison and union marshal

Make exact Equal and chunked comparison field-local too

Preserve nullability option in exact chunked comparison

Address review feedback on nullability-aware comparison

Compare record/table fields ignoring metadata

Address roborev findings on record/table field comparison

Avoid breaking arrow.Array with optional NullableMarshaler

Thread field-local nullability through list, union, and REE

Make run-end-encoded value comparison symmetric in ValueNullable