apache / apache/iceberg-rust

Validate the construction of `FileScanTaskDeleteFile`.

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

Description

We should validate this when building `FileScanTaskDeleteFile`.
```
isDv == FileScanTaskDeleteFile.content_type == PosDel && FileScanTaskDeleteFile.file_format == puffin
fn build() {
if (isDv) { validate_deletion_vector_task() }
}

```

_Originally posted by @CTTY in https://github.com/apache/iceberg-rust/pull/3035#discussion_r3919914422_

Contributor guide

Open the contributing guide

Research direction

Start at the FileScanTaskDeleteFile construction and build() entry point, then trace how isDv, content_type, file_format, PosDel, and puffin are represented. Determine where validate_deletion_vector_task() should be invoked and add coverage for the stated condition. Done means invalid deletion-vector task combinations are rejected during construction.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
data-engineering
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.