js8544 commented on issue #37276:
URL: https://github.com/apache/arrow/issues/37276#issuecomment-1689579326
Both tests assume a multi-threaded environment without checking the actual
threading capacity. Given that they intend to check the behavior of
multi-threaded execution, I think they should be skipped if there is only one
thread available:
```cpp
if (internal::GetCpuThreadPool()->GetCapacity() < 2) {
GTEST_SKIP() << "Test requires at least 2 threads";
}
```
The above patch works on my machine. Should I submit a PR for this? @pitrou
--
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]
