zanmato1984 opened a new pull request, #45923: URL: https://github.com/apache/arrow/pull/45923
### Rationale for this change The original test doesn't establish proper synchronization between `queue.WaitAndPop()` in one thread and the `fut_pop.get()` in the other. I.e., if the `std::async` thread (doing `queue.WaitAndPop()`) is scheduled by the os later than the timeout set for the `fut_pop.wait_for()` in the main thread, the timeout is returned and the assertion fires. ### What changes are included in this PR? Use `wait()` which will guarantee the synchronization to replace `wait_for()`. Of course this has the potential problem, say, if some bad change is made to the code, the test can hang forever. But I guess the timeout for the test in CI will aid. ### Are these changes tested? The fix is to the test. ### Are there any user-facing changes? None. -- 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]
