Skip to content

fix(array): recursively validate nested arrays#930

Open
fallintoplace wants to merge 1 commit into
apache:mainfrom
fallintoplace:fix/recursive-array-validation
Open

fix(array): recursively validate nested arrays#930
fallintoplace wants to merge 1 commit into
apache:mainfrom
fallintoplace:fix/recursive-array-validation

Conversation

@fallintoplace

Copy link
Copy Markdown
Contributor

What

Make top-level Validate and ValidateFull walk nested array data. Parent layouts, child counts and types, dictionary indices and values, and extension storage are checked before recursive validation. Errors include a path such as field "orders" -> list values -> field "name". Ordinary list and large-list offsets are now validated fully as well.

Tests

  • go test ./arrow/array -count=1
  • GOOS=linux GOARCH=386 go test -c -o /dev/null ./arrow/array
  • pre-commit run golangci-lint-full --all-files

Regression coverage includes malformed Struct -> List -> Struct -> String, dictionary values, and non-monotonic list offsets.

@fallintoplace fallintoplace requested a review from zeroshade as a code owner July 11, 2026 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant