Skip to content

fix(rest): reject blob offsets on non-Puffin delete files - #1613

Open
fallintoplace wants to merge 1 commit into
apache:mainfrom
fallintoplace:fix/rest-reject-nonpuffin-offsets
Open

fix(rest): reject blob offsets on non-Puffin delete files#1613
fallintoplace wants to merge 1 commit into
apache:mainfrom
fallintoplace:fix/rest-reject-nonpuffin-offsets

Conversation

@fallintoplace

Copy link
Copy Markdown
Contributor

What changed

Reject content-offset and content-size-in-bytes on position-delete files unless the file format is Puffin.

Why

These fields locate a deletion-vector blob inside a Puffin file. The decoder required them for Puffin files but accepted them on Parquet, Avro, and ORC position-delete files, creating an inconsistent DataFile.

Testing

  • Added cases for offset only, size only, and both fields on a non-Puffin position-delete file
  • Existing Puffin and equality-delete validation remains covered
  • go test ./catalog/rest

Signed-off-by: Minh Vu <vuhoangminh97@gmail.com>
@fallintoplace
fallintoplace requested a review from zeroshade as a code owner July 30, 2026 19:42
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