zeroshade opened a new pull request, #13158: URL: https://github.com/apache/arrow/pull/13158
The tests for [ARROW-16502](https://issues.apache.org/jira/browse/ARROW-16502) reused the same `StructBuilder` between cases. But since it was testing for a panic, it left the StructBuilder in a bad state. This is understandable as it's a panic. Because the order of Go map's is non-deterministic, the test would only intermittently fail if the panic test went first and succeed if it went second. By shifting the StructBuilder creating inside the test function, the two test cases no longer share it and the test is no longer flakey. -- 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]
