Skip to content

Core: Preserve delete file references for shared Puffin files - #17525

Open
mj006648 wants to merge 1 commit into
apache:mainfrom
mj006648:fix/delete-file-references
Open

Core: Preserve delete file references for shared Puffin files#17525
mj006648 wants to merge 1 commit into
apache:mainfrom
mj006648:fix/delete-file-references

Conversation

@mj006648

@mj006648 mj006648 commented Aug 5, 2026

Copy link
Copy Markdown

Summary

  • index delete files by location, content offset, and content size
  • preserve references when multiple deletion vectors share a Puffin file
  • add a round-trip regression test

Testing

  • ./gradlew :iceberg-core:test --tests org.apache.iceberg.rest.responses.TestPlanTableScanResponseParser --tests org.apache.iceberg.rest.responses.TestFetchScanTasksResponseParser --tests org.apache.iceberg.rest.responses.TestFetchPlanningResultResponseParser --no-daemon
  • ./gradlew spotlessCheck --no-daemon

Fixes #17334

@github-actions github-actions Bot added the core label Aug 5, 2026

@uros-b uros-b left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mj006648 is this duplicating #17497?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FixtureCatalog][Dev][Scan Planning] delete_file_references bug when 1 puffin file contains blobs for separate data files

2 participants