perf(test): Split HashJoinTest into two files to reduce test bottleneck

HashJoinTest.cpp (9,625 lines, ~807s) was the single heaviest test file,
dominating group0's execution time at ~13.6 min. Split it into:

- HashJoinTest.cpp (3,656 lines): MultiThreadedHashJoinTest (67 tests × 4
  params) plus interleaved HashJoinTest cases, ~525s estimated
- HashJoinTestSpill.cpp (6,041 lines): Remaining HashJoinTest cases
  including memory, lazy vectors, dynamic filters, spill, and memory
  reclaim tests (71 tests × 2 params), ~282s estimated

Both files share the HashJoinTestBase from utils/HashJoinTestBase.h.

Also rebalanced all 8 test groups with the new file. Max group time drops
from ~816s (13.6 min) to ~534s (8.9 min). All groups are now balanced at
~461-534s, which should reduce test phase wall time from ~20 min to ~9 min
with ctest -j 24.

Author: kgpai

velox/exec/tests/CMakeLists.txt

@@ -40,13 +40,12 @@ target_link_libraries(
 # bin-packing by estimated execution time. With VELOX_TESTS_PER_GROUP=10,
 # CMake takes files positionally in batches of 10 (files 1-10 = group0,
 # 11-20 = group1, etc.), so each batch of 10 must be independently balanced.
-# Group0 is dominated by HashJoinTest (~13.6 min); groups 1-7 are ~7 min each.
-# The previous alphabetical order concentrated slow tests (HashJoinTest,
-# IndexLookupJoinTest, HashTableTest) into the same groups, creating a
-# worst-case of ~18.8 min.
+# HashJoinTest was split into HashJoinTest (multi-threaded tests, ~525s) and
+# HashJoinTestSpill (single-threaded/spill tests, ~282s) to reduce the
+# bottleneck group from ~816s to ~534s. All groups are now ~462-534s (~8-9 min).
 set(
   VELOX_EXEC_TEST_SOURCES
-  # group0 (~816s): HashJoinTest is the single heaviest test; pad with light tests.
+  # group0 (~534s)
   HashJoinTest.cpp
   PrintPlanWithStatsTest.cpp
   ThreadDebugInfoTest.cpp
@@ -57,81 +56,82 @@ set(
   ValuesTest.cpp
   ParallelProjectTest.cpp
   AsyncConnectorTest.cpp
-  # group1 (~424s)
+  # group1 (~463s)
   IndexLookupJoinTest.cpp
-  PrefixSortTest.cpp
-  HashPartitionFunctionTest.cpp
-  ExpandTest.cpp
-  SplitListenerTest.cpp
+  MergeTest.cpp
+  FilterProjectTest.cpp
+  ScaleWriterLocalPartitionTest.cpp
+  CountingJoinTest.cpp
+  CustomJoinTest.cpp
   AssignUniqueIdTest.cpp
-  HashBitRangeTest.cpp
+  TaskListenerTest.cpp
   SplitTest.cpp
-  ArrowStreamTest.cpp
   SqlTest.cpp
-  # group2 (~423s)
+  # group2 (~462s)
+  HashJoinTestSpill.cpp
+  OperatorTraceTest.cpp
+  OrderByTest.cpp
+  PrefixSortTest.cpp
+  MarkDistinctTest.cpp
+  MergerTest.cpp
+  StreamingEnforceDistinctTest.cpp
+  MixedUnionWithTableScanTest.cpp
+  HashBitRangeTest.cpp
+  WindowFunctionRegistryTest.cpp
+  # group3 (~462s)
   TableScanTest.cpp
-  AggregationTest.cpp
-  MergeTest.cpp
-  LimitTest.cpp
-  TopNTest.cpp
-  SpatialIndexTest.cpp
-  PlanNodeSerdeTest.cpp
+  OutputBufferManagerTest.cpp
+  NestedLoopJoinTest.cpp
+  MixedUnionTest.cpp
+  VectorHasherTest.cpp
+  ContainerRowSerdeTest.cpp
+  ExpandTest.cpp
+  FunctionResolutionTest.cpp
   SpillStatsTest.cpp
-  WindowFunctionRegistryTest.cpp
-  PlanNodeToSummaryStringTest.cpp
-  # group3 (~422s)
+  AggregateFunctionRegistryTest.cpp
+  # group4 (~462s)
   MergeJoinTest.cpp
-  OperatorTraceTest.cpp
-  LocalPartitionTest.cpp
-  SortBufferTest.cpp
-  ScaleWriterLocalPartitionTest.cpp
-  ContainerRowSerdeTest.cpp
+  HashTableTest.cpp
+  StreamingAggregationTest.cpp
+  PartitionedOutputTest.cpp
+  TopNRowNumberTest.cpp
+  TopNTest.cpp
   EnforceSingleRowTest.cpp
-  MixedUnionWithTableScanTest.cpp
+  FilterToExpressionTest.cpp
   ColumnStatsCollectorTest.cpp
-  AggregateFunctionRegistryTest.cpp
-  # group4 (~422s)
+  RoundRobinPartitionFunctionTest.cpp
+  # group5 (~462s)
   MultiFragmentTest.cpp
-  OutputBufferManagerTest.cpp
-  StreamingAggregationTest.cpp
-  MixedUnionTest.cpp
-  VectorHasherTest.cpp
-  CountingJoinTest.cpp
-  MergerTest.cpp
-  ConcatFilesSpillMergeStreamTest.cpp
+  ExchangeClientTest.cpp
+  AggregationTest.cpp
+  MemoryReclaimerTest.cpp
+  LimitTest.cpp
+  UnnestTest.cpp
+  SplitListenerTest.cpp
+  HilbertIndexTest.cpp
   AddressableNonNullValueListTest.cpp
-  RoundRobinPartitionFunctionTest.cpp
-  # group5 (~421s)
+  ArrowStreamTest.cpp
+  # group6 (~462s)
   RowContainerTest.cpp
-  HashTableTest.cpp
-  NestedLoopJoinTest.cpp
-  MemoryReclaimerTest.cpp
+  SpillTest.cpp
+  LocalPartitionTest.cpp
+  HashJoinBridgeTest.cpp
   RowNumberTest.cpp
   EnforceDistinctTest.cpp
-  MarkSortedTest.cpp
   TraceUtilTest.cpp
-  FilterToExpressionTest.cpp
+  PlanNodeSerdeTest.cpp
   ScaledScanControllerTest.cpp
-  # group6 (~421s)
+  PlanNodeToSummaryStringTest.cpp
+  # group7 (~461s)
   TableWriterTest.cpp
-  ExchangeClientTest.cpp
-  OrderByTest.cpp
-  HashJoinBridgeTest.cpp
-  FilterProjectTest.cpp
-  UnnestTest.cpp
-  StreamingEnforceDistinctTest.cpp
-  FunctionResolutionTest.cpp
-  TaskListenerTest.cpp
-  PlanNodeToStringTest.cpp
-  # group7 (~419s)
   SpillerTest.cpp
-  SpillTest.cpp
   WindowTest.cpp
-  PartitionedOutputTest.cpp
-  TopNRowNumberTest.cpp
-  MarkDistinctTest.cpp
-  CustomJoinTest.cpp
-  HilbertIndexTest.cpp
+  SortBufferTest.cpp
+  HashPartitionFunctionTest.cpp
+  SpatialIndexTest.cpp
+  MarkSortedTest.cpp
+  ConcatFilesSpillMergeStreamTest.cpp
+  PlanNodeToStringTest.cpp
 )
 
 set(