GH-43116: [C++][Compute] Mark KeyCompare.CompareColumnsToRowsLarge as large memory test

[zanmato1984]

Rationale for this change

This test consumes more than 4GB memory and causes oom-kill when running with TSAN as reported in #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: [CI][C++] thread-sanitizer job fails on arrow-compute-internals-test #43116

[github-actions]

⚠️ GitHub issue #43116 has been automatically assigned in GitHub to PR creator.

[zanmato1984]

Hi @pitrou , mind to take a look? Thanks.

cc @raulcd

[zanmato1984]

@github-actions crossbow submit -g cpp

[github-actions]

Revision: 85bf559

Submitted crossbow builds: ursacomputing/crossbow @ actions-5c19a21694

Task	Status
test-alpine-linux-cpp
test-build-cpp-fuzz
test-conda-cpp
test-conda-cpp-valgrind
test-cuda-cpp
test-debian-12-cpp-amd64
test-debian-12-cpp-i386
test-fedora-39-cpp
test-ubuntu-20.04-cpp
test-ubuntu-20.04-cpp-bundled
test-ubuntu-20.04-cpp-minimal-with-formats
test-ubuntu-20.04-cpp-thread-sanitizer
test-ubuntu-22.04-cpp
test-ubuntu-22.04-cpp-20
test-ubuntu-22.04-cpp-emscripten
test-ubuntu-22.04-cpp-no-threading
test-ubuntu-24.04-cpp
test-ubuntu-24.04-cpp-gcc-14

[zanmato1984]

@github-actions crossbow submit -g cpp

[github-actions]

Revision: ff8e08c

Submitted crossbow builds: ursacomputing/crossbow @ actions-f080521a0f

Task	Status
test-alpine-linux-cpp
test-build-cpp-fuzz
test-conda-cpp
test-conda-cpp-valgrind
test-cuda-cpp
test-debian-12-cpp-amd64
test-debian-12-cpp-i386
test-fedora-39-cpp
test-ubuntu-20.04-cpp
test-ubuntu-20.04-cpp-bundled
test-ubuntu-20.04-cpp-minimal-with-formats
test-ubuntu-20.04-cpp-thread-sanitizer
test-ubuntu-22.04-cpp
test-ubuntu-22.04-cpp-20
test-ubuntu-22.04-cpp-emscripten
test-ubuntu-22.04-cpp-no-threading
test-ubuntu-24.04-cpp
test-ubuntu-24.04-cpp-gcc-14

[zanmato1984]

@github-actions crossbow submit -g cpp

[zanmato1984]

@github-actions crossbow submit -g cpp

[github-actions]

Revision: a1a1ccb

Submitted crossbow builds: ursacomputing/crossbow @ actions-0257529eff

Task	Status
test-alpine-linux-cpp
test-build-cpp-fuzz
test-conda-cpp
test-conda-cpp-valgrind
test-cuda-cpp
test-debian-12-cpp-amd64
test-debian-12-cpp-i386
test-fedora-39-cpp
test-ubuntu-20.04-cpp
test-ubuntu-20.04-cpp-bundled
test-ubuntu-20.04-cpp-minimal-with-formats
test-ubuntu-20.04-cpp-thread-sanitizer
test-ubuntu-22.04-cpp
test-ubuntu-22.04-cpp-20
test-ubuntu-22.04-cpp-emscripten
test-ubuntu-22.04-cpp-no-threading
test-ubuntu-24.04-cpp
test-ubuntu-24.04-cpp-gcc-14

[zanmato1984]

@github-actions crossbow submit -g cpp

[github-actions]

Revision: c81724a

Submitted crossbow builds: ursacomputing/crossbow @ actions-3eff9d8bd6

Task	Status
test-alpine-linux-cpp
test-build-cpp-fuzz
test-conda-cpp
test-conda-cpp-valgrind
test-cuda-cpp
test-debian-12-cpp-amd64
test-debian-12-cpp-i386
test-fedora-39-cpp
test-ubuntu-20.04-cpp
test-ubuntu-20.04-cpp-bundled
test-ubuntu-20.04-cpp-minimal-with-formats
test-ubuntu-20.04-cpp-thread-sanitizer
test-ubuntu-22.04-cpp
test-ubuntu-22.04-cpp-20
test-ubuntu-22.04-cpp-emscripten
test-ubuntu-22.04-cpp-no-threading
test-ubuntu-24.04-cpp
test-ubuntu-24.04-cpp-gcc-14

[zanmato1984]

@github-actions crossbow submit -g cpp

[github-actions]

Revision: aed0361

Submitted crossbow builds: ursacomputing/crossbow @ actions-07ab3e3101

Task	Status
test-alpine-linux-cpp
test-build-cpp-fuzz
test-conda-cpp
test-conda-cpp-valgrind
test-cuda-cpp
test-debian-12-cpp-amd64
test-debian-12-cpp-i386
test-fedora-39-cpp
test-ubuntu-20.04-cpp
test-ubuntu-20.04-cpp-bundled
test-ubuntu-20.04-cpp-minimal-with-formats
test-ubuntu-20.04-cpp-thread-sanitizer
test-ubuntu-22.04-cpp
test-ubuntu-22.04-cpp-20
test-ubuntu-22.04-cpp-emscripten
test-ubuntu-22.04-cpp-no-threading
test-ubuntu-24.04-cpp
test-ubuntu-24.04-cpp-gcc-14

[raulcd]

LGTM
@felipecrv @bkietz are you ok for me merging this for the release?

[felipecrv]

LGTM @felipecrv @bkietz are you ok for me merging this for the release?

Sure. Go ahead.

[conbench-apache-arrow]

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 5b5c164.

There were 2 benchmark results indicating a performance regression:

• Commit Run on test-mac-arm at 2024-07-05 10:28:09Z
  • tpch (R) with engine=arrow, format=parquet, language=R, memory_map=False, query_id=TPCH-21, scale_factor=1
  • tpch (R) with engine=arrow, format=native, language=R, memory_map=False, query_id=TPCH-21, scale_factor=1

The full Conbench report has more details. It also includes information about 10 possible false positives for unstable benchmarks that are known to sometimes produce them.