jorgecarleitao commented on a change in pull request #9114:
URL: https://github.com/apache/arrow/pull/9114#discussion_r560369235
##########
File path: rust/arrow/src/error.rs
##########
@@ -24,6 +24,9 @@ use std::error::Error;
/// Many different operations in the `arrow` crate return this error type.
#[derive(Debug)]
pub enum ArrowError {
+ /// "Not yet implemented" Arrow error.
+ /// Returned when functionality is not yet available.
+ NYI(String),
Review comment:
I think that the error name should be explicit, like all others. I can't
understand what `NYI` means without having to go to the docs, and in an error
message, the person often has no access to the docs (at least not in a 1 click
thing).
`NotYetImplemented`?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
