apache/fluss-rust

[rust] Add a SnapshotId type alias

Aperta

#636 aperta il 24 giu 2026

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Rust (48 fork)auto 404
good first issue

Metriche repository

Star
 (54 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Search before asking

  • I searched in the 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!

Guida contributor

Direzione di ricerca
Esamina gli alias di tipo esistenti in lib.rs (ad esempio, TableId) e aggiungi `pub type SnapshotId = i64;`. Quindi aggiorna tutti gli usi hardcoded di `i64` relativi agli ID snapshot nei file specificati: LakeSnapshot in metadata/table.rs, KvSnapshotLeaseForBucket in metadata/kv snapshot lease.rs, e i costruttori di richiesta get kv snapshot metadata / get lake snapshot.
Tech stack
rust
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85