apache / apache/fluss-rust

[rust] Add a SnapshotId type alias

Open Beginner friendly
#636 1 comment 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
Rust
Stars
58
Forks
50
PR merge metrics
No merged PRs in 30d

Description

### Search before asking

- [x] I searched in the [issues](https://github.com/apache/fluss-rust/issues) and found nothing similar.

### Description

We have TableId, PartitionId, and BucketId aliases in lib.rs, but snapshot_id is still spelled i64 everywhere. Adding a matching SnapshotId alias would keep things consistent and make signatures easier to read.

A few hand-written sites to update: LakeSnapshot (metadata/table.rs), KvSnapshotLeaseForBucket (metadata/kv_snapshot_lease.rs), and the get_kv_snapshot_metadata / get_lake_snapshot request constructors.

### Willingness to contribute

- [ ] I'm willing to submit a PR!

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in lib.rs to inspect the existing TableId, PartitionId, and BucketId aliases, then review SnapshotId uses in metadata/table.rs and metadata/kv_snapshot_lease.rs. Check the get_kv_snapshot_metadata and get_lake_snapshot request constructors; done means the named snapshot_id sites use a consistent SnapshotId alias.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
api
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
78/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.