jorisvandenbossche commented on code in PR #38972:
URL: https://github.com/apache/arrow/pull/38972#discussion_r1412395202
##########
cpp/src/arrow/filesystem/test_util.h:
##########
@@ -170,7 +170,7 @@ class ARROW_TESTING_EXPORT GenericFileSystemTest {
virtual bool allow_move_dir_over_non_empty_dir() const { return false; }
// - Whether the filesystem allows appending to a file
virtual bool allow_append_to_file() const { return true; }
- // - Whether the filesystem allows appending to a new (not existent yet) file
+ // - Whether the filesystem allows appending to a new (not yet extant) file
Review Comment:
I am not a native speak, but I already work in english context for more than
10 years, and to be honest I hadn't heard of that word .. ;-) So I would prefer
if we can use something simpler. Is "not yet existing" also correct English?
--
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
