tpboudreau commented on a change in pull request #6993:
URL: https://github.com/apache/arrow/pull/6993#discussion_r411569025
##########
File path: cpp/src/arrow/util/io_util_test.cc
##########
@@ -446,6 +446,56 @@ TEST(CreateDirTree, Basics) {
ASSERT_OK_AND_ASSIGN(fn, temp_dir->path().Join("EF"));
ASSERT_OK_AND_ASSIGN(created, CreateDirTree(fn));
ASSERT_TRUE(created);
+
+#ifndef __APPLE__
Review comment:
I experienced failures in the CI pipeline on macOS and I was unable to
locate clear documentation of the path name limits (I'm not a macOS expert).
I figured it might be best to separately address macOS in another issue, if
there's community support for that. (This patch leaves macOS builds and tests
unchanged.)
If you believe this test should run as is under macOS, I'll remove the
#ifndef and follow up on any issues.
----------------------------------------------------------------
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:
us...@infra.apache.org
