This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/orc.git
The following commit(s) were added to refs/heads/main by this push:
new 4ff03de02 ORC-2003: Upgrade `guava` to 33.5.0-jre
4ff03de02 is described below
commit 4ff03de0213505d09edac733d277c5e9ae403d53
Author: sychen <[email protected]>
AuthorDate: Tue Sep 23 21:21:07 2025 -0700
ORC-2003: Upgrade `guava` to 33.5.0-jre
### What changes were proposed in this pull request?
This PR aims to upgrade `guava` to 33.5.0-jre.
### Why are the changes needed?
https://github.com/apache/orc/actions/runs/17914579352/job/50933668060?pr=2410
```
TestSerializationUtils.testSubtractionOverflowGuava:138 expected:
<overflow: checkedSubtract(22222222222, -9223372036854775808)> but was: <long
overflow>
```
https://github.com/google/guava/commit/fec36dc383ac3388bfcb1decb1cb94ba63eabad2
### How was this patch tested?
GA
### Was this patch authored or co-authored using generative AI tooling?
No
Closes #2413 from cxzl25/ORC-2003.
Authored-by: sychen <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
java/core/src/test/org/apache/orc/impl/TestSerializationUtils.java | 6 +++---
java/pom.xml | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/java/core/src/test/org/apache/orc/impl/TestSerializationUtils.java
b/java/core/src/test/org/apache/orc/impl/TestSerializationUtils.java
index 206306513..cda818893 100644
--- a/java/core/src/test/org/apache/orc/impl/TestSerializationUtils.java
+++ b/java/core/src/test/org/apache/orc/impl/TestSerializationUtils.java
@@ -135,21 +135,21 @@ public class TestSerializationUtils {
LongMath.checkedSubtract(22222222222L, Long.MIN_VALUE);
fail("expected ArithmeticException for overflow");
} catch (ArithmeticException ex) {
- assertEquals("overflow: checkedSubtract(22222222222,
-9223372036854775808)", ex.getMessage());
+ assertEquals("long overflow", ex.getMessage());
}
try {
LongMath.checkedSubtract(-22222222222L, Long.MAX_VALUE);
fail("expected ArithmeticException for overflow");
} catch (ArithmeticException ex) {
- assertEquals("overflow: checkedSubtract(-22222222222,
9223372036854775807)", ex.getMessage());
+ assertEquals("long overflow", ex.getMessage());
}
try {
LongMath.checkedSubtract(Long.MIN_VALUE, Long.MAX_VALUE);
fail("expected ArithmeticException for overflow");
} catch (ArithmeticException ex) {
- assertEquals("overflow: checkedSubtract(-9223372036854775808,
9223372036854775807)", ex.getMessage());
+ assertEquals("long overflow", ex.getMessage());
}
assertEquals(-8106206116692740190L,
diff --git a/java/pom.xml b/java/pom.xml
index 8572cbdb7..28ac87a29 100644
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -221,7 +221,7 @@
<dependency>
<groupId>com.google.guava</groupId>
<artifactId>guava</artifactId>
- <version>33.4.0-jre</version>
+ <version>33.5.0-jre</version>
<scope>test</scope>
</dependency>
<dependency>