Skip to content

Commit 4f39e6e

Browse files
authored
GH-40507: [C++][ORC] Upgrade ORC to 2.0.0 (#40508)
### Rationale for this change This PR aims to upgrade to a new major version of Apache ORC: https://orc.apache.org/news/2024/03/08/ORC-2.0.0/ ### What changes are included in this PR? This PR upgrades ORC dependency from 1.9.2 to 2.0.0. ### Are these changes tested? Pass the CIs. ### Are there any user-facing changes? No. * GitHub Issue: #40507 Lead-authored-by: Antoine Pitrou <antoine@python.org> Co-authored-by: Gang Wu <ustcwg@gmail.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
1 parent cf832b8 commit 4f39e6e

4 files changed

Lines changed: 14 additions & 4 deletions

File tree

ci/scripts/python_wheel_macos_build.sh

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,13 @@ echo "=== (${PYTHON_VERSION}) Building Arrow C++ libraries ==="
9191
: ${VCPKG_FEATURE_FLAGS:=-manifests}
9292
: ${VCPKG_TARGET_TRIPLET:=${VCPKG_DEFAULT_TRIPLET:-x64-osx-static-${CMAKE_BUILD_TYPE}}}
9393

94+
echo "=== Protobuf compiler versions on PATH ==="
95+
which -a protoc || echo "no protoc on PATH!"
96+
97+
echo "=== Protobuf compiler version from vcpkg ==="
98+
_pbc=${VCPKG_ROOT}/installed/${VCPKG_TARGET_TRIPLET}/tools/protobuf/protoc
99+
echo "$_pbc: `$_pbc --version`"
100+
94101
mkdir -p ${build_dir}/build
95102
pushd ${build_dir}/build
96103

@@ -122,6 +129,7 @@ cmake \
122129
-DARROW_SUBSTRAIT=${ARROW_SUBSTRAIT} \
123130
-DARROW_TENSORFLOW=${ARROW_TENSORFLOW} \
124131
-DARROW_USE_CCACHE=ON \
132+
-DARROW_VERBOSE_THIRDPARTY_BUILD=ON \
125133
-DARROW_WITH_BROTLI=${ARROW_WITH_BROTLI} \
126134
-DARROW_WITH_BZ2=${ARROW_WITH_BZ2} \
127135
-DARROW_WITH_LZ4=${ARROW_WITH_LZ4} \
@@ -134,7 +142,6 @@ cmake \
134142
-DCMAKE_INSTALL_PREFIX=${build_dir}/install \
135143
-DCMAKE_OSX_ARCHITECTURES=${CMAKE_OSX_ARCHITECTURES} \
136144
-DCMAKE_UNITY_BUILD=${CMAKE_UNITY_BUILD} \
137-
-DORC_PROTOBUF_EXECUTABLE=${VCPKG_ROOT}/installed/${VCPKG_TARGET_TRIPLET}/tools/protobuf/protoc \
138145
-DORC_SOURCE=BUNDLED \
139146
-DPARQUET_REQUIRE_ENCRYPTION=${PARQUET_REQUIRE_ENCRYPTION} \
140147
-DVCPKG_MANIFEST_MODE=OFF \

ci/scripts/python_wheel_manylinux_build.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,6 @@ cmake \
123123
-DCMAKE_INSTALL_LIBDIR=lib \
124124
-DCMAKE_INSTALL_PREFIX=/tmp/arrow-dist \
125125
-DCMAKE_UNITY_BUILD=${CMAKE_UNITY_BUILD} \
126-
-DORC_PROTOBUF_EXECUTABLE=${VCPKG_ROOT}/installed/${VCPKG_TARGET_TRIPLET}/tools/protobuf/protoc \
127126
-DORC_SOURCE=BUNDLED \
128127
-DPARQUET_REQUIRE_ENCRYPTION=${PARQUET_REQUIRE_ENCRYPTION} \
129128
-DVCPKG_MANIFEST_MODE=OFF \

cpp/thirdparty/versions.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,8 @@ ARROW_OPENTELEMETRY_BUILD_VERSION=v1.8.1
9090
ARROW_OPENTELEMETRY_BUILD_SHA256_CHECKSUM=3d640201594b07f08dade9cd1017bd0b59674daca26223b560b9bb6bf56264c2
9191
ARROW_OPENTELEMETRY_PROTO_BUILD_VERSION=v0.17.0
9292
ARROW_OPENTELEMETRY_PROTO_BUILD_SHA256_CHECKSUM=f269fbcb30e17b03caa1decd231ce826e59d7651c0f71c3b28eb5140b4bb5412
93-
ARROW_ORC_BUILD_VERSION=1.9.2
94-
ARROW_ORC_BUILD_SHA256_CHECKSUM=7f46f2c184ecefd6791f1a53fb062286818bd8710c3f08b94dd3cac365e240ee
93+
ARROW_ORC_BUILD_VERSION=2.0.0
94+
ARROW_ORC_BUILD_SHA256_CHECKSUM=9107730919c29eb39efaff1b9e36166634d1d4d9477e5fee76bfd6a8fec317df
9595
ARROW_PROTOBUF_BUILD_VERSION=v21.3
9696
ARROW_PROTOBUF_BUILD_SHA256_CHECKSUM=2f723218f6cb709ae4cdc4fb5ed56a5951fc5d466f0128ce4c946b8c78c8c49f
9797
# Because of https://github.com/Tencent/rapidjson/pull/1323, we require

dev/tasks/python-wheels/github.osx.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,10 @@ jobs:
4747
brew install bash bison coreutils ninja
4848
echo "$(brew --prefix bison)/bin" >> $GITHUB_PATH
4949
50+
- name: Homebrew packages
51+
run: |
52+
brew list
53+
5054
- name: Retrieve VCPKG version from arrow/.env
5155
run: |
5256
vcpkg_version=$(cat "arrow/.env" | grep "VCPKG" | cut -d "=" -f2 | tr -d '"')

0 commit comments

Comments
 (0)