GH-48376: [C++] Update GoogleTest from 1.16.0 to 1.17.0

[tennisleng]

Rationale for this change

This updates GoogleTest to version 1.17.0 to prevent potential undefined behavior in ASSERT_DOUBLE_EQ and ASSERT_FLOAT_EQ macros.

Prior to version 1.17, GoogleTest used union-based type punning which is undefined behavior in C++ (reading from a union member that wasn't most recently written).

See: google/googletest@59c924b

What changes are included in this PR?

Updated cpp/thirdparty/versions.txt:

• ARROW_GTEST_BUILD_VERSION: 1.16.0 -> 1.17.0
• ARROW_GTEST_BUILD_SHA256_CHECKSUM: Updated to match new version

Are these changes tested?

This is a dependency version update. Existing tests will validate compatibility.

Are there any user-facing changes?

No. This updates GoogleTest to version 1.17.0 to prevent potential undefined behavior in ASSERT_DOUBLE_EQ and ASSERT_FLOAT_EQ macros.

Prior to version 1.17, GoogleTest used union-based type punning which is undefined behavior in C++ (reading from a union member that wasn't most recently written). This was fixed in GoogleTest commit 59c924bc.

See: google/googletest@59c924b

• GitHub Issue: [C++] Update the version of GoogleTest in cpp/thirdparty/versions.txt to 1.17 to prevent potential undefined behavior in ASSERT_DOUBLE_EQ and ASSERT_FLOAT_EQ #48376

[github-actions]

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

[github-actions]

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

[zanmato1984]

+1

[zanmato1984]

Thanks for this!

@kou what do you think?

[zanmato1984]

@github-actions crossbow submit -g cpp

[github-actions]

Revision: e60782e

Submitted crossbow builds: ursacomputing/crossbow @ actions-31a1fe2b5e

Task	Status
example-cpp-minimal-build-static
example-cpp-minimal-build-static-system-dependency
example-cpp-tutorial
test-build-cpp-fuzz
test-conda-cpp
test-conda-cpp-valgrind
test-cuda-cpp-ubuntu-22.04-cuda-11.7.1
test-cuda-cpp-ubuntu-24.04-cuda-13.0.2
test-debian-12-cpp-amd64
test-debian-12-cpp-i386
test-fedora-42-cpp
test-ubuntu-22.04-cpp
test-ubuntu-22.04-cpp-20
test-ubuntu-22.04-cpp-bundled
test-ubuntu-22.04-cpp-emscripten
test-ubuntu-22.04-cpp-no-threading
test-ubuntu-24.04-cpp
test-ubuntu-24.04-cpp-bundled-offline
test-ubuntu-24.04-cpp-gcc-13-bundled
test-ubuntu-24.04-cpp-gcc-14
test-ubuntu-24.04-cpp-minimal-with-formats
test-ubuntu-24.04-cpp-thread-sanitizer

[github-actions]

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

[github-actions]

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

[kou]

+1

[conbench-apache-arrow]

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

There were no benchmark performance regressions. 🎉

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.