Skip to content

Use bounded equality-delete reads - #27

Merged
nicosuave merged 1 commit into
mainfrom
bounded-equality-delete-dependency
Aug 18, 2026
Merged

Use bounded equality-delete reads#27
nicosuave merged 1 commit into
mainfrom
bounded-equality-delete-dependency

Conversation

@nicosuave

@nicosuave nicosuave commented Aug 18, 2026

Copy link
Copy Markdown
Member

Summary

  • point dlt-iceberg at the Sidequery PyIceberg fork with incremental equality-delete reads
  • refresh the lockfile to the exact fork commit

Validation

  • uv run pytest -q: 174 passed, 16 skipped
  • Docker Lakekeeper/MinIO integration: 100,000 initial rows merged with 50,000 composite-key rows to 125,000 final rows in one snapshot

Packaging note

This keeps the existing VCS dependency model and therefore is not itself publishable to public PyPI. The failed 0.7.0 workflow must be repaired separately by publishing the fork as a normal distribution or waiting for the required upstream functionality.

@nicosuave
nicosuave marked this pull request as ready for review August 18, 2026 04:29
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@nicosuave
nicosuave merged commit ad70abe into main Aug 18, 2026
1 check passed
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