apacheGH-43116: [C++][Compute] Mark KeyCompare.CompareColumnsToRowsLa…

…rge as large memory test (apache#43128) ### Rationale for this change This test consumes more than 4GB memory and causes oom-kill when running with TSAN as reported in apache#43116 . ### What changes are included in this PR? Limit its running by marking it as large memory test. ### Are these changes tested? Change is test. ### Are there any user-facing changes? None. * GitHub Issue: apache#43116 Authored-by: Ruoxi Sun <[email protected]> Signed-off-by: Raúl Cumplido <[email protected]>
zanmato1984 · Jul 9, 2024 · cd066b5 · cd066b5
1 parent 4f3b3ab
commit cd066b5
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/cpp/src/arrow/compute/row/compare_test.cc b/cpp/src/arrow/compute/row/compare_test.cc
@@ -166,10 +166,9 @@ TEST(KeyCompare, CompareColumnsToRowsTempStackUsage) {
   }
 }
 
-#ifndef ARROW_VALGRIND
 // Compare columns to rows at offsets over 2GB within a row table.
 // Certain AVX2 instructions may behave unexpectedly causing troubles like GH-41813.
-TEST(KeyCompare, CompareColumnsToRowsLarge) {
+TEST(KeyCompare, LARGE_MEMORY_TEST(CompareColumnsToRowsLarge)) {
   if constexpr (sizeof(void*) == 4) {
     GTEST_SKIP() << "Test only works on 64-bit platforms";
   }
@@ -300,7 +299,6 @@ TEST(KeyCompare, CompareColumnsToRowsLarge) {
               num_rows);
   }
 }
-#endif  // ARROW_VALGRIND
 
 }  // namespace compute
 }  // namespace arrow