vibhatha commented on PR #12672: URL: https://github.com/apache/arrow/pull/12672#issuecomment-1111822395
> There's still test failures. Can we just do the 'dumb' thing that seems to work? > > https://github.com/apache/arrow/blob/dc97883dee25ba8da55c7591060c44de2ea00865/cpp/src/parquet/test_util.cc#L42-L72 > > Or else looks like we have to fix `arrow::internal::PlatformFilename` under MinGW. @lidavidm It seems like the path issue is related to Substrait internals. How we handle the path is not compatible with Windows. For this PR going to skip windows and create a follow up PR to fix the issue. cc @westonpace Filed the JIRA: https://issues.apache.org/jira/browse/ARROW-16392 Will be working on this after merging this PR. We can use the existing test case to evaluate it and enable the test for windows too. -- 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
