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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org