-
Notifications
You must be signed in to change notification settings - Fork 506
Comparing changes
Open a pull request
base repository: apache/orc
base: v2.1.3
head repository: apache/orc
compare: v2.1.4
- 12 commits
- 24 files changed
- 6 contributors
Commits on Jul 5, 2025
-
Preparing branch for 2.1.4 development
Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for cfb7313 - Browse repository at this point
Copy the full SHA cfb7313View commit details
Commits on Jul 15, 2025
-
ORC-1952: [C++] Fix the issue where the value of headerThirdByte exce…
…eds the valid byte range ### What changes were proposed in this pull request? Ensure DIRECT encoding is employed when the input value exceeds the valid byte range. ### Why are the changes needed? In Patched Base Encoding, the value of headerThirdByte exceeds the valid byte range. ### How was this patch tested? Add the RleV2_value_limit_test to the RleTest suite. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2324 from luffy-zh/ORC-1952. Lead-authored-by: luffy-zh <[email protected]> Co-authored-by: ffacs <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]> (cherry picked from commit 3d61594) Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for ddedb96 - Browse repository at this point
Copy the full SHA ddedb96View 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 c94b11d - Browse repository at this point
Copy the full SHA c94b11dView commit details
Commits on Sep 12, 2025
-
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.1` branch CIs are broken like the following. - #2384 - https://github.com/apache/orc/actions/runs/17682608443/job/50260045180?pr=2384 ``` 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 #2386 from dongjoon-hyun/ORC-1973-2.1. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 6f5638f - Browse repository at this point
Copy the full SHA 6f5638fView commit details -
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 #2384 from dongjoon-hyun/ORC-1996-2.1. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for dcb577c - Browse repository at this point
Copy the full SHA dcb577cView 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.1` CI is broken on `MacOS` due to this. - https://github.com/apache/orc/tree/branch-2.1 - https://github.com/apache/orc/actions/runs/17683046777/job/50261415209 ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2387 from dongjoon-hyun/ORC-1974-2.1. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 6129e4f - Browse repository at this point
Copy the full SHA 6129e4fView 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 #2391 from dongjoon-hyun/ORC-1892-2.1. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 89d37ea - Browse repository at this point
Copy the full SHA 89d37eaView 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 #2395 from dongjoon-hyun/ORC-1893-2.1. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 8e238cc - Browse repository at this point
Copy the full SHA 8e238ccView 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 c4420c2 - Browse repository at this point
Copy the full SHA c4420c2View commit details
Commits on Oct 16, 2025
-
ORC-2027: [C++] Fix undefined behavior in DoubleColumnReader::readFlo…
…at() ### What changes were proposed in this pull request? Unaligned reads are UB in C++. We can not guarantee that the `bufferPointer_` pointer is aligned by `alignof(int32_t)`. ### How was this patch tested? Use UBsan to test in private repo. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #2444 from HuaHuaY/fix_issue_2027. Lead-authored-by: Zehua Zou <[email protected]> Co-authored-by: Zehua Zou <[email protected]> Signed-off-by: Gang Wu <[email protected]> (cherry picked from commit c4fa9fa) Signed-off-by: Gang Wu <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 2be451b - Browse repository at this point
Copy the full SHA 2be451bView 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.1` 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 #2467 from dongjoon-hyun/ORC-2041-2.1. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: William Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 365c605 - Browse repository at this point
Copy the full SHA 365c605View commit details
Commits on Jan 6, 2026
-
Signed-off-by: William Hyun <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 2efceb3 - Browse repository at this point
Copy the full SHA 2efceb3View 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.1.3...v2.1.4