fix(arrow/array): Parity with Arrow C++ when reading/writing `null` to/from JSON - #833

Open

serramatutu wants to merge 22 commits into

apache:mainfrom

serramatutu:serramatutu/JSON-nulls-v2

Commits on Sep 2, 2026

`RecordBuilder.UnmarshalOne()` checks for non-nullable fields
serramatutu
committed
a026359
`StructBuilder.UnmarshalOne()` checks for non-nullable fields
serramatutu
committed
379e86f
Make struct and record respect `field.Nullable` when serializing
serramatutu
committed
115d4f6
Add stricter tests to null JSON in record and struct
serramatutu
committed
7353862
Fix tests that were implicitly depending on wrong nullable semantics
serramatutu
committed
5743041
Fix invalid JSON literal in RecordBuilder test
serramatutu
committed
c576f1b
Fix unreachable nullable-field branch in Struct.GetOneForMarshal
serramatutu
committed
4f771fb
Validate before appending in StructBuilder.UnmarshalOne
serramatutu
committed
81295cb
Rely on builder checkpoints to discard failed RecordBuilder rows
serramatutu
committed
7e6570b
Make `StructBuilder` error when required field is not given
serramatutu
committed
83e5d59
Declare nullable fields in rollback tests that decode null
serramatutu
committed
138fdcb
Roll back StructBuilder rows with builder checkpoints
serramatutu
committed
0946848
Check FromJSON error before releasing the roundtripped array
serramatutu
committed
a93bb86
Add `unmarshalListValues`, check the schema of JSON list items
serramatutu
committed
59f1711
add `unmarshalChild` helper and make list builder use it
serramatutu
committed
e58088c
Make map builder use validating list builder
serramatutu
committed
cc6ea82
Make union builder respect inner nullability
serramatutu
committed
42cfc61
Make REE builder respect nullability
serramatutu
committed
24a5e49
Add checkpoints to standalone builders
serramatutu
committed
515b176
Fix broken assertion
serramatutu
committed
2efa1f6
Fix performance issues with JSON decoding
serramatutu
committed
8d37a55
Collapse nested field builder into `nestedJSONDecoder`
serramatutu
committed
b2595af

Navigation Menu

Uh oh!

fix(arrow/array): Parity with Arrow C++ when reading/writing `null` to/from JSON - #833

fix(arrow/array): Parity with Arrow C++ when reading/writing `null` to/from JSON#833
serramatutu wants to merge 22 commits into
apache:mainfrom
serramatutu:serramatutu/JSON-nulls-v2

Commits on Sep 2, 2026

`RecordBuilder.UnmarshalOne()` checks for non-nullable fields

`StructBuilder.UnmarshalOne()` checks for non-nullable fields

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

Add stricter tests to null JSON in record and struct

Fix tests that were implicitly depending on wrong nullable semantics

Fix invalid JSON literal in RecordBuilder test

Fix unreachable nullable-field branch in Struct.GetOneForMarshal

Validate before appending in StructBuilder.UnmarshalOne

Rely on builder checkpoints to discard failed RecordBuilder rows

Make `StructBuilder` error when required field is not given

Declare nullable fields in rollback tests that decode null

Roll back StructBuilder rows with builder checkpoints

Check FromJSON error before releasing the roundtripped array

Add `unmarshalListValues`, check the schema of JSON list items

add `unmarshalChild` helper and make list builder use it

Make map builder use validating list builder

Make union builder respect inner nullability

Make REE builder respect nullability

Add checkpoints to standalone builders

Fix broken assertion

Fix performance issues with JSON decoding

Collapse nested field builder into `nestedJSONDecoder`