Rename position delete index to clarify referenced data file key
Maintainers usually reply within 4 days
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 82/100
Research direction
Start in crates/iceberg/src/delete_file_index.rs and read the PopulatedDeleteFileIndex field, its local bindings, references, and comments. Rename pos_deletes_by_path to pos_deletes_by_referenced_data_file consistently, preserving behavior; run the existing delete-file index tests, formatting, and Clippy to verify the change.
Written by the indexing model from the issue text.
Description
Is your feature request related to a problem or challenge?
PopulatedDeleteFileIndex::pos_deletes_by_path is keyed by the referenced data file's path. The current name leaves it unclear whether the key is the delete file's own path or the data file it applies to.
This tracks the naming follow-up requested in the #3035 review. In the September 21 review of #3145, mbutrovich explicitly asked for a separate tracking issue. The validation work in #3145 can remain separate.
Describe the solution you'd like
Rename the private field and matching local bindings from pos_deletes_by_path to pos_deletes_by_referenced_data_file in crates/iceberg/src/delete_file_index.rs, updating references and comments consistently. Preserve delete-file applicability, indexing, and scan behavior.
Validation should include the existing delete-file index tests, formatting, and Clippy. This is an internal naming cleanup with no intended public API or behavior change.
Willingness to contribute
I would be willing to contribute to this feature with guidance from the Iceberg Rust community.
Prepared with AI assistance; this issue records the requested follow-up and does not claim an implementation.
- Dominant language
- Rust
- Stars
- 1.4k
- Forks
- 592
- Avg merge
- 5d 12h
- Merged PRs (30d)
- 59
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from apache/iceberg-rust
-
V1 manifest lists written by Java lose added/existing/deleted file countsPossibly taken @kacper-li-airspace-intelligence claimed this 1 day ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
apache/iceberg-rust#3445 ·
Maintainers usually reply within 4 days
-
Row filter keeps every row for `<` and `<=` on a null column that is missing from the data fileOpen
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
apache/iceberg-rust#3355 · 1 reaction ·
Maintainers usually reply within 4 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
apache/iceberg-rust#3348 ·
Maintainers usually reply within 4 days
-
datafusion
Difficulty 1/5 Under an hour Newbie friendliness 88/100
apache/iceberg-rust#3324 · 1 reaction ·
Maintainers usually reply within 4 days
-
Remove license clarification for zstd-sys once workspace upgrades zstd (zstd 0.14, zstd-sys 2.1)Open
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
apache/iceberg-rust#3307 ·
Maintainers usually reply within 4 days
All issues in apache/iceberg-rust
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
dependencies effort:low flow:chore rust
Difficulty 2/5 1-3 hours Newbie friendliness 64/100
jaroslawherod/micold-ai-ide#875 ·
Maintainers usually reply within 1 day
-
priority: low
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
storytold/photocraft#6286 · 1 comment ·
Maintainers usually reply within 1 day
-
Four problem loader browser tests fail on WindowsPossibly taken @Patriciaath999 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
Maintainers usually reply within 1 day
-
Kazakh (kk_KZ) and Georgian (ka_GE) system locales get English instead of the shipped translationsOpenbug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day