Olawoyin007 opened a new pull request, #657: URL: https://github.com/apache/fluss-rust/pull/657
### Purpose Linked issue: close #636 `TableId`, `PartitionId`, and `BucketId` already have type aliases in `lib.rs` (extended to the write path in #635), but `snapshot_id` was still spelled `i64` everywhere. This adds a matching `SnapshotId` alias so snapshot-related signatures read consistently with the other ID types. ### Brief change log - Add `pub type SnapshotId = i64;` next to the existing aliases in `lib.rs`. - Apply it to the hand-written sites: `LakeSnapshot` (`metadata/table.rs`), `LakeSnapshotInfo`, `KvSnapshot`, `KvSnapshotLeaseForBucket`, the admin client methods (`get_kv_snapshot_metadata`, `get_lake_snapshot`), and the `GetKvSnapshotMetadataRequest` / `GetLakeSnapshotRequest` constructors. - Generated proto code is untouched; `HashMap<TableBucket, i64>` offset values stay `i64` since they are offsets, not snapshot IDs. ### Tests No behavior change (the alias resolves to the same type). `cargo test --workspace`: 576 passed, 0 failed. `cargo clippy --workspace --all-targets` and `cargo fmt --all --check` are clean. ### API and Format Public signatures now name `SnapshotId` instead of `i64`; since it is a plain type alias, this is source- and ABI-compatible. No storage format impact. ### Documentation No new feature; no documentation changes needed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
