Copilot commented on code in PR #416: URL: https://github.com/apache/fluss-rust/pull/416#discussion_r2876724729
########## crates/fluss/src/row/binary/mod.rs: ########## @@ -19,7 +19,7 @@ mod binary_writer; pub use binary_writer::*; -/// The binary row format types, it indicates the generated [`BinaryRow`] type by the [`BinaryWriter`] +/// The binary row format types, it indicates the generated row type by the [`BinaryWriter`] Review Comment: The doc sentence is grammatically awkward (“The binary row format types, it indicates…”). Consider rewriting to a clear single sentence (e.g., that the enum variants indicate which row representation `BinaryWriter` produces). ```suggestion /// Indicates which binary row format [`BinaryWriter`] generates. ``` ########## crates/fluss/src/lib.rs: ########## @@ -15,6 +15,93 @@ // specific language governing permissions and limitations // under the License. +//! Apache Fluss (Incubating) Official Rust Client +//! +//! Official Rust client library for [Apache Fluss (Incubating)](https://fluss.apache.org/). +//! It supports **primary key (KV) tables** (upsert + lookup) and **log tables** (append + scan). Review Comment: The PR description still contains the placeholder “Linked issue: close #xxx” from the template. Please either link the actual issue or remove the placeholder text so the PR metadata is accurate. -- 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]
