jsoref commented on code in PR #38972:
URL: https://github.com/apache/arrow/pull/38972#discussion_r1414205387
##########
cpp/src/arrow/filesystem/azurefs.cc:
##########
@@ -335,7 +335,7 @@ class ObjectInputFile final : public io::RandomAccessFile {
}
return internal::ExceptionToStatus(
"GetProperties failed for '" + blob_client_->GetUrl() +
- "' with an unexpected Azure error. Can not initialise an
ObjectInputFile "
+ "' with an unexpected Azure error. Cannot initialise an
ObjectInputFile "
Review Comment:
The problem with the current text is that it's hard to know if it's saying
`Can <choose> not-<to>-initialise an ObjectInputFile` or if it's saying `Cannot
initialise an ObjectInputFile` -- that ambiguity is dangerous and ambiguity
should be avoided (especially when it's incredibly cheap, as here).
--
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]
