You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@fluss.apache.org by "gnuhpc (via GitHub)" <gi...@apache.org> on 2026/06/19 01:09:07 UTC

[PR] [rust] Sync Fluss 1.x proto definitions and ApiKey registry [fluss-rust]

gnuhpc opened a new pull request, #628:
URL: https://github.com/apache/fluss-rust/pull/628

   ## What
   
   Syncs the Rust client's `flüss_api.proto` and `ApiKey` registry with Fluss 1.x, and updates pre-existing message wrappers that reference the renamed symbols so the crate still builds.
   
   - Add all 1.x message types (ACLs, KV snapshots, producer offsets, cluster config, rebalance, server tags, etc.) and new optional fields (`rack`, `remote_data_dir`, `leader_epoch`, `agg_mode`, ...).
   - Register 24 new `ApiKey` variants (1023–1064) in `api_key.rs`.
   - Update `build.rs` prost-bytes config for the new proto fields.
   - Add `None` defaults in `convert.rs` / `partition.rs`.
   - Update pre-existing message wrappers that reference renamed proto fields / ApiKey variants so the build stays green:
     - `create_partition.rs`: `ignore_if_exists` -> `ignore_if_not_exists` (same wire field 3; matches Java's end-to-end value flow, no inversion)
     - `get_latest_lake_snapshot.rs`: `ApiKey::GetLatestLakeSnapshot` -> `GetLakeSnapshot` (same id 1032)
     - `list_databases.rs`: populate new `include_summary` field
     - `lookup.rs`: `PbLookupReqForBucket.key` -> `keys`; new `LookupRequest` fields
   
   ## Why
   
   Foundation for the rest of the 1.x support stack. Each commit in this stacked PR series must leave `cargo build` green, so the proto/contract change ships together with the caller updates it requires.
   
   ## Notes
   
   This is PR 1/6 of a stacked series (`pr/1` -> `pr/2` -> ... -> `pr/6`). The field renames are same-tag, so they are wire-compatible; the Java server's end-to-end handling was verified to preserve the original `ignore_if_exists` semantic.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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: issues-unsubscribe@fluss.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [rust] Sync Fluss 1.x proto definitions and ApiKey registry [fluss-rust]

Posted by "fresh-borzoni (via GitHub)" <gi...@apache.org>.
fresh-borzoni merged PR #628:
URL: https://github.com/apache/fluss-rust/pull/628


-- 
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: issues-unsubscribe@fluss.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org