Skip to content

Commit

Permalink
GH-35636: [C++] Extract two expensive test suites from compute-vector…
Browse files Browse the repository at this point in the history
…-test (#36401)

### Rationale for this change

`arrow-compute-vector-test` is too big and takes a long time to run because of that.

### What changes are included in this PR?

Extracting two tests.

Timings on my machine (Debug builds with ASAN).

```
debug/arrow-compute-vector-test > /dev/null  11.54s user 0.47s system 99% cpu 12.023 total
debug/arrow-compute-vector-sort-test > /dev/null  13.30s user 0.26s system 99% cpu 13.579 total
debug/arrow-compute-vector-selection-test > /dev/null  6.97s user 0.22s system 99% cpu 7.207 total
```

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.

* Closes: #35636

Authored-by: Felipe Oliveira Carvalho <felipekde@gmail.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
  • Loading branch information
felipecrv committed Jul 3, 2023
1 parent f951f0c commit 1bfa241
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions cpp/src/arrow/compute/kernels/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -73,12 +73,15 @@ add_arrow_compute_test(vector_test
vector_hash_test.cc
vector_nested_test.cc
vector_replace_test.cc
vector_selection_test.cc
vector_sort_test.cc
vector_run_end_encode_test.cc
select_k_test.cc
test_util.cc)

add_arrow_compute_test(vector_sort_test SOURCES vector_sort_test.cc test_util.cc)

add_arrow_compute_test(vector_selection_test SOURCES vector_selection_test.cc
test_util.cc)

add_arrow_benchmark(vector_hash_benchmark PREFIX "arrow-compute")
add_arrow_benchmark(vector_sort_benchmark PREFIX "arrow-compute")
add_arrow_benchmark(vector_partition_benchmark PREFIX "arrow-compute")
Expand Down

0 comments on commit 1bfa241

Please sign in to comment.