[THRIFT-4106] concurrency_test fails randomly

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 0.10.0
• Fix Version/s: 0.11.0
• Component/s: C++ - Library

Environment

MinGW (appveyor), travis CI

• Assignee: James E. King III
• Reporter: James E. King III
• Created: 01/Mar/17 03:46
• Updated: 14/Dec/17 13:55
• Resolved: 07/Sep/17 17:14

Description

While adding Appveyor build support for MinGW (THRIFT-4081), this test failed periodically. It would throw an exception in ThreadFactoryTest reapTest where it calls monitor.wait(1000). It is reproducible locally as well if you have msys2/mingw64 and can use the instructions in the msys2 readme in build/cmake. The test has been disabled in mingw appveyor builds for now (those builds are new...)

Travis CI builds are also showing an occasional failure in the test.

Need to root cause, fix, and re-enable in any CI builds where it was disabled.

Issue Links

• relates to: THRIFT-4081 Provide a MinGW 64-bit Appveyor CI build for better pull request validation
• links to: GitHub Pull Request #1343