jsoref commented on code in PR #38972:
URL: https://github.com/apache/arrow/pull/38972#discussion_r1414245813
##########
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:
I've been making contributions like these for years (probably over a decade
really...), I don't mind pushback and I understand the concerns. I spent a
dozen years working on a large project where I did a nontrivial amount of
cvs-archeology (even once the project was no longer using CVS...).
These days,
https://github.blog/changelog/2022-03-24-ignore-commits-in-the-blame-view-beta/
means it's easier for devs to skip past commits like this (and at this point,
arrow already has quite a few of my commits because this project requested that
pieces be split to aid in reviewing/merging). -- Once these PRs are all merged,
I'd be happy to create a PR to add `.git-blame-ignore-revs` with all of the
related commits if people are interested.
--
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]
