apache / apache/iceberg-rust

[EPIC] Remove all pub field accessor.

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

Description

### What's the feature are you trying to implement?

With more and more features introduced, we found one common requirement: we need to ensure that when a struct instance has been built, we need to ensure that it's valid. Some structs expose pub field accessors directly, which makes maintaining the guarantee infeasible. We need to remove all pub accessors of those struct.

### Willingness to contribute

None

Contributor guide

Open the contributing guide

Research direction

No files, tests, or entry points are named. Start by inventorying the structs that expose public field accessors and determine the scope of the epic from the existing Rust APIs. Done means the targeted public accessors have been removed so struct validity can be maintained after construction.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend-api-design
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.