apache/fluss-rust

[rust] Add a SnapshotId type alias

Aberta

#636 aberto em 24 de jun. de 2026

Ver no GitHub
 (1 comentário) (0 reação) (0 responsável)Rust (48 forks)auto 404
good first issue

Métricas do repositório

Stars
 (54 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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!

Guia do colaborador

Direção de pesquisa
Examine os aliases de tipo existentes em lib.rs (por exemplo, TableId) e adicione `pub type SnapshotId = i64;`. Em seguida, atualize todos os usos hardcoded de `i64` relacionados a IDs de snapshot nos arquivos especificados: LakeSnapshot em metadata/table.rs, KvSnapshotLeaseForBucket em metadata/kv snapshot lease.rs, e os construtores de requisição get kv snapshot metadata / get lake snapshot.
Pilha de tecnologia
rust
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RustGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85