apache / apache/iceberg-rust

[EPIC] Type safe TableProperties parsing.

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

Description

### Is your feature request related to a problem or challenge?

Currently when we parse table properties, we parse them manully. Ideally we should be able to pasre `TableProperties` using derived approach as following:
```rust
pub struct TableProperties {
#[key="read.split.target-size"]
#[default(134217728// or some const)]
#[doc("Target size when combining data input splits")]
read_split_size: usize,
....
}
```

### Describe the solution you'd like

_No response_

### Willingness to contribute

I can contribute to this feature independently

Contributor guide

Open the contributing guide

Research direction

Start by locating the current manual parsing implementation for TableProperties and trace how its properties are represented and validated. Compare that flow with the proposed derived approach and identify the required fields, defaults, and documentation; done means TableProperties can be parsed type-safely without losing existing behavior, with tests covering the resulting properties.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
databases
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.