alamb commented on code in PR #9139:
URL: https://github.com/apache/arrow-rs/pull/9139#discussion_r2682087288
##########
parquet-variant/src/variant/object.rs:
##########
@@ -390,7 +390,8 @@ impl<'m, 'v> VariantObject<'m, 'v> {
/// Returns the value of the field with the specified name, if any.
///
- /// `Ok(None)` means the field does not exist; `Err` means the search
encountered an error.
+ /// Returns `Some(Variant)` if the field exists, or `None` if the field
does not exist
+ /// or if the lookup fails safely.
Review Comment:
Rust also uses the word `safe` to mean particular things about memory so I
agree removing the safely word would make this less confusing
--
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]
