Skip to content

fix: return protobuf map key conversion errors#909

Open
fallintoplace wants to merge 2 commits into
apache:mainfrom
fallintoplace:fix-protobuf-map-key-error-clean
Open

fix: return protobuf map key conversion errors#909
fallintoplace wants to merge 2 commits into
apache:mainfrom
fallintoplace:fix-protobuf-map-key-error-clean

Conversation

@fallintoplace

Copy link
Copy Markdown
Contributor

Summary

  • Make protobuf map key conversion return an error instead of panicking for unsupported key kinds.
  • Thread conversion errors through map iteration into record construction with contextual message including field name and map type.
  • Add regression tests for invalid map key conversion and contextual append error path.

Validation

  • Changes committed in branch: fix-protobuf-map-key-error-clean
  • Target: apache/arrow-go:main
  • No tests run in this step.

@fallintoplace fallintoplace requested a review from zeroshade as a code owner July 5, 2026 17:35
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