jnturton commented on PR #2554: URL: https://github.com/apache/drill/pull/2554#issuecomment-1132459141
Hi @nielsbasjes don't worry, everyone was a bit mystified by this one! Your PR didn't introduce the bug, it only revealed an existing bug via an accident. It _seems_ like the unit test that @vdiravka fixed here was only ever working because a race condition affecting test initialisation kept turning out in its favour by good luck. We think that by adding some unrelated tests, your PR shifted Maven / Surefire's overall unit test scheduling so that this test's run of luck came to an end and its buggy initialisation was revealed. -- 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]
