zanmato1984 commented on code in PR #40237:
URL: https://github.com/apache/arrow/pull/40237#discussion_r1503065582
##########
cpp/src/arrow/scalar_test.cc:
##########
@@ -1984,4 +1985,41 @@ TEST_F(TestExtensionScalar, ValidateErrors) {
AssertValidationFails(scalar);
}
+template <typename T>
+class TestScalarScratchSpace : public ::testing::Test {
+ public:
+ TestScalarScratchSpace() = default;
+};
+
+TYPED_TEST_SUITE(TestScalarScratchSpace, BaseBinaryOrBinaryViewLikeArrowTypes);
+
+// GH-40069: race condition when filling the scratch space of a scalar in
parallel.
Review Comment:
I didn't see a TSAN CI job in current job list. Either I'm not sure how to
trigger one.
But I confirm that the `ctest` command failed in my local, built with
`-DARROW_USE_TSAN=ON -DARROW_JEMALLOC=OFF -DARROW_MIMALLOC=OFF`.
What's interesting, though I'm not exactly sure why, is that gtest actually
reported PASSED because the TSAN error is just WARNING printed to stderr. But
the test program did aborted (returned non-zero code) and ctest considered it
failed.
--
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]
