-
Notifications
You must be signed in to change notification settings - Fork 506
Comparing changes
Open a pull request
base repository: apache/orc
base: v2.0.6
head repository: apache/orc
compare: v2.0.7
- 14 commits
- 20 files changed
- 3 contributors
Commits on Jul 4, 2025
-
Preparing branch for 2.0.7 development
Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for f5f76ee - Browse repository at this point
Copy the full SHA f5f76eeView commit details
Commits on Aug 8, 2025
-
ORC-1970: [CI] Fix cpp-linter-action to use hash tag
### What changes were proposed in this pull request? Bump cpp-linter-action version to f91c446a32ae3eb9f98fef8c9ed4c7cb613a4f8a. ### Why are the changes needed? Fix CI. ### How was this patch tested? The CI job could test it. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2354 from luffy-zh/ORC-1970. Authored-by: luffy-zh <[email protected]> Signed-off-by: Gang Wu <[email protected]> (cherry picked from commit 7dc7178) Signed-off-by: Gang Wu <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 0ab01b4 - Browse repository at this point
Copy the full SHA 0ab01b4View commit details
Commits on Sep 12, 2025
-
ORC-1996: Remove
MacOS 13from GitHub Action CI and docs### What changes were proposed in this pull request? This PR aims to remove `MacOS 13` from GitHub Action CI and docs. ### Why are the changes needed? https://github.blog/changelog/2025-07-11-upcoming-changes-to-macos-hosted-runners-macos-latest-migration-and-xcode-support-policy-updates/ > The macOS 13 hosted runner image is closing down, following our [N-1 OS support policy](https://github.com/actions/runner-images?tab=readme-ov-file#software-and-image-support). This process will begin October 1, 2025, and the image will be fully retired on December 4, 2025. We recommend updating workflows to use ### How was this patch tested? Check the CIs triggered on this PR. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2385 from dongjoon-hyun/ORC-1996-2.0. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 487e484 - Browse repository at this point
Copy the full SHA 487e484View commit details -
ORC-1973: [C++] Use
int64_tinstead ofgoogle::protobuf::int64fo……r `Protobuf` v22+ ### What changes were proposed in this pull request? This PR aims to use `int64_t` instead of `google::protobuf::int64` to follow ProtocolBuff v22.0 changes. - https://github.com/protocolbuffers/protobuf/releases/tag/v22.0 > Fixed C++ code generation for protos that use int32_t, uint32_t, int64_t, uint64_t, size_t as field names. ### Why are the changes needed? Currently, `branch-2.0` branch CIs are broken like the following. - https://github.com/apache/orc/actions/runs/17683570590/job/50263141762 ``` In file included from /Users/runner/work/orc/orc/c++/src/io/InputStream.cc:19: /Users/runner/work/orc/orc/c++/src/io/InputStream.hh:75:31: error: no type named 'int64' in namespace 'google::protobuf' virtual google::protobuf::int64 ByteCount() const override; ~~~~~~~~~~~~~~~~~~^ /Users/runner/work/orc/orc/c++/src/io/InputStream.cc:115:21: error: no type named 'int64' in namespace 'google::protobuf' google::protobuf::int64 SeekableArrayInputStream::ByteCount() const { ~~~~~~~~~~~~~~~~~~^ /Users/runner/work/orc/orc/c++/src/io/InputStream.cc:116:42: error: no type named 'int64' in namespace 'google::protobuf' return static_cast<google::protobuf::int64>(position); ~~~~~~~~~~~~~~~~~~^ 3 errors generated. ``` ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2388 from dongjoon-hyun/ORC-1973-2.0. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for be61206 - Browse repository at this point
Copy the full SHA be61206View commit details -
ORC-1974: [C++] Use
google::protobuf::TextFormatinstead of `DebugS……tring` for `Protobuf` v30+ ### What changes were proposed in this pull request? This PR aims to use `google::protobuf::TextFormat` instead of `DebugString` for Protobuf v30+. - https://github.com/protocolbuffers/protobuf/releases/tag/v30.0 > Make DebugString print debug output, enable debug markers for debug output ### Why are the changes needed? Otherwise, our C++ `orc-metadata` tool will have a regression to expose the debug marker string, `goo.gle/debugstr`. Currently, `branch-2.0` CI is broken on `MacOS` due to this. - https://github.com/apache/orc/actions/runs/17683726166/job/50263664096 > +goo.gle/debugonly ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2390 from dongjoon-hyun/ORC-1974-2.0. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 39a7b6d - Browse repository at this point
Copy the full SHA 39a7b6dView commit details -
ORC-1825: [C++] Update Snappy to 1.2.1
### What changes were proposed in this pull request? This PR aims to update Snappy to 1.2.1. ### Why are the changes needed? To use the most recent version of Snappy. - https://github.com/google/snappy/releases/tag/1.1.8 - https://github.com/google/snappy/releases/tag/1.1.9 - https://github.com/google/snappy/releases/tag/1.1.10 - https://github.com/google/snappy/releases/tag/1.2.0 - https://github.com/google/snappy/releases/tag/1.2.1 ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2392 from dongjoon-hyun/ORC-1825-2.0. Authored-by: William Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for fc3afab - Browse repository at this point
Copy the full SHA fc3afabView commit details -
ORC-1892: [C++] Upgrade
snappyto 1.2.2### What changes were proposed in this pull request? This PR aims to upgrade `snappy` to 1.2.2 in `cmake_modules`. ### Why are the changes needed? To bring the latest bug fixes. - https://github.com/google/snappy/releases/tag/1.2.2 > We fixed a very old issue of data corruption when compressed size exceeds 4GB. This can happen when you compress data close to 4GB and it's incompressible, for example, random data. ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2393 from dongjoon-hyun/ORC-1892-2.0. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 2e7ea11 - Browse repository at this point
Copy the full SHA 2e7ea11View commit details -
ORC-1827: [C++] Update ZLIB to 1.3.1
### What changes were proposed in this pull request? This PR aims to update ZLIB to 1.3.1. ### Why are the changes needed? To use the latest version. - https://www.zlib.net/ChangeLog.txt ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2394 from dongjoon-hyun/ORC-1827-2.0. Authored-by: William Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for ad53677 - Browse repository at this point
Copy the full SHA ad53677View commit details -
ORC-1893: [C++] Upgrade
zstdto 1.5.7### What changes were proposed in this pull request? This PR aims to upgrade `zstd` to 1.5.7 in `cmake_modules` module. `GitHub Action MacOS 15` is updated to ZSTD 1.5.7. - https://github.com/actions/runner-images/blob/macos-15-arm64/20250911.2324/images/macos/macos-15-arm64-Readme.md ### Why are the changes needed? To bring the latest bug fixes, - https://github.com/facebook/zstd/releases/tag/v1.5.6 - https://github.com/facebook/zstd/releases/tag/v1.5.7 ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2396 from dongjoon-hyun/ORC-1893-2.0. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 680af6c - Browse repository at this point
Copy the full SHA 680af6cView commit details -
ORC-1828: [C++] Update LZ4 to 1.10.0
### What changes were proposed in this pull request? This PR aims to update LZ4 to 1.10.0. ### Why are the changes needed? To use the latest version. - https://github.com/lz4/lz4/releases/tag/v1.9.4 - https://github.com/lz4/lz4/releases/tag/v1.10.0 In addition, we need to recover the CI because currently `branch-2.0` CI is broken due to `LZ4` compilation failure like the following. - https://github.com/apache/orc/actions/runs/17686548024/job/50272495675 ``` [ 25%] Performing configure step for 'lz4_ep' CMake Error at /Users/runner/work/orc/orc/build/lz4_ep-prefix/src/lz4_ep-stamp/lz4_ep-configure-RELWITHDEBINFO.cmake:49 (message): Command failed: 1 '/opt/homebrew/bin/cmake' '-DCMAKE_INSTALL_PREFIX=/Users/runner/work/orc/orc/build/c++/libs/thirdparty/lz4_ep-install' '-DCMAKE_INSTALL_LIBDIR=lib' '-DBUILD_SHARED_LIBS=OFF' '-GUnix Makefiles' '-S' '/Users/runner/work/orc/orc/build/lz4_ep-prefix/src/lz4_ep/build/cmake' '-B' '/Users/runner/work/orc/orc/build/lz4_ep-prefix/src/lz4_ep-build' See also /Users/runner/work/orc/orc/build/lz4_ep-prefix/src/lz4_ep-stamp/lz4_ep-configure-*.log make[2]: *** [lz4_ep-prefix/src/lz4_ep-stamp/lz4_ep-configure] Error 1 make[1]: *** [CMakeFiles/lz4_ep.dir/all] Error 2 make: *** [all] Error 2 ``` ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2397 from dongjoon-hyun/ORC-1828-2.0. Authored-by: William Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 42a9dba - Browse repository at this point
Copy the full SHA 42a9dbaView commit details -
ORC-1896: Add
CMAKE_POLICY_VERSION_MINIMUM=3.12to `ThirdpartyToolc……hain.cmake` ### What changes were proposed in this pull request? This PR aims to add `CMAKE_POLICY_VERSION_MINIMUM=3.12` explicitly to `ThirdpartyToolchain.cmake` for Apache ORC 2.0.x. ### Why are the changes needed? The minimum requirement of CMake is `3.12.0` in Apache ORC 2.0.x. https://github.com/apache/orc/blob/42a9dbacc6acfcfddbb378a12bdc7c03e38590c8/CMakeLists.txt#L18 This will resolve the `branch-2.0` CI failure. - https://github.com/apache/orc/actions/runs/17686769349/job/50273163640 ``` CMake Error at /Users/runner/work/orc/orc/build/protobuf_ep-prefix/src/protobuf_ep-stamp/protobuf_ep-configure-RELWITHDEBINFO.cmake:49 (message): Command failed: 1 '/opt/homebrew/bin/cmake' '-DCMAKE_INSTALL_PREFIX=/Users/runner/work/orc/orc/build/c++/libs/thirdparty/protobuf_ep-install' '-DCMAKE_INSTALL_LIBDIR=lib' '-DBUILD_SHARED_LIBS=OFF' '-Dprotobuf_BUILD_TESTS=OFF' '-GUnix Makefiles' '-S' '/Users/runner/work/orc/orc/build/protobuf_ep-prefix/src/protobuf_ep/cmake' '-B' '/Users/runner/work/orc/orc/build/protobuf_ep-prefix/src/protobuf_ep-build' See also /Users/runner/work/orc/orc/build/protobuf_ep-prefix/src/protobuf_ep-stamp/protobuf_ep-configure-*.log make[2]: *** [protobuf_ep-prefix/src/protobuf_ep-stamp/protobuf_ep-configure] Error 1 make[1]: *** [CMakeFiles/protobuf_ep.dir/all] Error 2 make: *** [all] Error 2 ``` ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2398 from dongjoon-hyun/ORC-1896-2.0. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for ddd2ee7 - Browse repository at this point
Copy the full SHA ddd2ee7View commit details
Commits on Sep 26, 2025
-
ORC-2010: Use
IANAIdentifierAmerica/Los_Angelesinstead of `US/……Pacific` in Java ### What changes were proposed in this pull request? This PR aims to use `IANA` Identifier `America/Los_Angeles` instead of `US/Pacific` which is removed from `TZDB`. ### Why are the changes needed? To make Apache ORC code up-to-date with the current `IANA` zone pattern and be robust from the underlying OS changes. ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2418 from dongjoon-hyun/ORC-2010. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]> (cherry picked from commit 2c023ef) Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 1dbcb07 - Browse repository at this point
Copy the full SHA 1dbcb07View commit details
Commits on Jan 3, 2026
-
ORC-2041: Upgrade
cpp-linter-actionhash to match ASF infra### What changes were proposed in this pull request? This PR aims to. update `cpp-linter-action` hash to match ASF infra. ### Why are the changes needed? Currently, the AS-IS `branch-2.0` CI is broken due to the ASF INFRA change. > The action cpp-linter/cpp-linter-actionf91c446a32ae3eb9f98fef8c9ed4c7cb613a4f8a is not allowed in apache/orc because all actions must be from a repository owned by your enterprise, created by GitHub, verified in the GitHub Marketplace, or match one of the patterns: ... As of now, the value is the following. - https://github.com/apache/infrastructure-actions/blob/main/actions.yml#L278 ``` cpp-linter/cpp-linter-action: 0f6d1b8d7e38b584cbee606eb23d850c217d54f8: tag: v2.15.1 ``` ### How was this patch tested? Pass the CIs on this PR. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2468 from dongjoon-hyun/ORC-2041-2.0. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: William Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for d056e8c - Browse repository at this point
Copy the full SHA d056e8cView commit details
Commits on Jan 5, 2026
-
Signed-off-by: William Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 79d554e - Browse repository at this point
Copy the full SHA 79d554eView commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v2.0.6...v2.0.7