apache/fluss-rust

[rust] Add a SnapshotId type alias

Ouverte

#636 ouverte le 24 juin 2026

Voir sur GitHub
 (1 commentaire) (0 réaction) (0 personne assignée)Rust (48 forks)auto 404
good first issue

Métriques du dépôt

Stars
 (54 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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!

Guide contributeur

Direction de recherche
Examinez les alias de type existants dans lib.rs (par exemple, TableId) et ajoutez `pub type SnapshotId = i64;`. Mettez ensuite à jour toutes les utilisations codées en dur de `i64` liées aux identifiants de snapshot dans les fichiers spécifiés : LakeSnapshot dans metadata/table.rs, KvSnapshotLeaseForBucket dans metadata/kv snapshot lease.rs, et les constructeurs de requêtes get kv snapshot metadata / get lake snapshot.
Stack technique
rust
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85