GH-41581: [C++][CMake] correctly use Protobuf_PROTOC_EXECUTABLE

[h-vetinari]

Fixes #41581

• GitHub Issue: [C++][CMake] handle Protobuf_PROTOC_EXECUTABLE correctly in cross-compilation #41581

[github-actions]

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

In the case of PARQUET issues on JIRA the title also supports:

PARQUET-${JIRA_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

See also:

• Other pull requests
• Contribution Guidelines - Contributing Overview

[github-actions]

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

[assignUser]

@github-actions crossbow submit -g cpp r-binary-packages

[github-actions]

Revision: ec001f1

Submitted crossbow builds: ursacomputing/crossbow @ actions-ff0c327b41

Task	Status
r-binary-packages
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]

👍
Thanks @h-vetinari

[h-vetinari]

Gentle ping here :)

[assignUser]

ah sorry I lost track of this, Thanks!

[conbench-apache-arrow]

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

There was 1 benchmark result indicating a performance regression:

• Commit Run on ursa-i9-9960x at 2024-05-15 16:00:57Z
  • tpch (R) with engine=arrow, format=native, language=R, memory_map=False, query_id=TPCH-17, scale_factor=1

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