Copilot commented on code in PR #450: URL: https://github.com/apache/fluss-rust/pull/450#discussion_r2972500926
########## website/docs/user-guide/rust/api-reference.md: ########## @@ -35,7 +35,7 @@ Complete API reference for the Fluss Rust client. | Method | Description | |-------------------------------------------------------------------------------|------------------------------------------------| | `async fn new(config: Config) -> Result<Self>` | Create a new connection to a Fluss cluster | -| `async fn get_admin(&self) -> Result<FlussAdmin>` | Get the admin interface for cluster management | +| `fn get_admin(&self) -> Result<FlussAdmin>` | Get the admin interface for cluster management | Review Comment: The Rust API reference lists `get_admin` as returning `Result<FlussAdmin>`, but the actual public signature is `fn get_admin(&self) -> Result<Arc<FlussAdmin>>` (see `crates/fluss/src/client/connection.rs`). Please update this doc row to match the real return type so readers don’t try to use it as an owned `FlussAdmin` value. ```suggestion | `fn get_admin(&self) -> Result<Arc<FlussAdmin>>` | Get the admin interface for cluster management | ``` -- 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]
