apache / apache/iceberg-rust

Add `validate_added_delete_files`

Open
#2,517 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
1.4k
Forks
567
Avg merge
2d 2h
Merged PRs (30d)
93

Description

> Java has a `validateDeleteFileForVersion` which might be nice to have the same? https://github.com/apache/iceberg/blob/0f657edf12dc29f8487a679bfdd4210e9588d014/core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java#L295-L316

_Originally posted by @xanderbailey in [#2367](https://github.com/apache/iceberg-rust/pull/2367/changes#r3200237452)_

We should also think about guarding validations using configurable flags, so they can be skipped when needed

Contributor guide

Open the contributing guide

Research direction

Start by comparing Apache Iceberg Java's MergingSnapshotProducer.validateDeleteFileForVersion implementation at the linked lines, then locate the corresponding Rust snapshot and delete-file validation entry points. Confirm the intended validate_added_delete_files behavior and whether configurable flags are required; done means the expected validation is covered and the skip behavior is settled.

Written by the indexing model from the issue text.

Assessment

Tech stack
java, rust
Domain
data
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.