This is an automated email from the ASF dual-hosted git repository.
chaokunyang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/fury.git
The following commit(s) were added to refs/heads/main by this push:
new 15096f57 fix(java): DeserializationException#getMessage call (#1830)
15096f57 is described below
commit 15096f576df9896d0f724a0563b68b6f38842b34
Author: Haemin Yoo <[email protected]>
AuthorDate: Sat Sep 7 14:29:03 2024 +0900
fix(java): DeserializationException#getMessage call (#1830)
## What does this PR do?
Fix failure on `DeserializationException#getMessage`
```
java.lang.StringIndexOutOfBoundsException: Range [311, 2) out of bounds for
length 313
at java.base/jdk.internal.util.Preconditions$1.apply(Unknown Source)
at java.base/jdk.internal.util.Preconditions$1.apply(Unknown Source)
at java.base/jdk.internal.util.Preconditions$4.apply(Unknown Source)
at java.base/jdk.internal.util.Preconditions$4.apply(Unknown Source)
at java.base/jdk.internal.util.Preconditions.outOfBounds(Unknown
Source)
at
java.base/jdk.internal.util.Preconditions.outOfBoundsCheckFromToIndex(Unknown
Source)
at
java.base/jdk.internal.util.Preconditions.checkFromToIndex(Unknown Source)
at java.base/java.lang.AbstractStringBuilder.delete(Unknown Source)
at java.base/java.lang.StringBuilder.delete(Unknown Source)
at
org.apache.fury.exception.DeserializationException.getMessage(DeserializationException.java:59)
```
## Related issues
N/A
## Does this PR introduce any user-facing change?
N/A
## Benchmark
N/A
---
.../main/java/org/apache/fury/exception/DeserializationException.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/java/fury-core/src/main/java/org/apache/fury/exception/DeserializationException.java
b/java/fury-core/src/main/java/org/apache/fury/exception/DeserializationException.java
index 562d18b0..7af5139f 100644
---
a/java/fury-core/src/main/java/org/apache/fury/exception/DeserializationException.java
+++
b/java/fury-core/src/main/java/org/apache/fury/exception/DeserializationException.java
@@ -56,7 +56,7 @@ public class DeserializationException extends FuryException {
for (Object readObject : readObjects) {
builder.append(readObject == null ? null :
readObject.getClass()).append(", ");
}
- builder.delete(builder.length() - 2, 2);
+ builder.delete(builder.length() - 2, builder.length());
builder.append("]");
return builder.toString();
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
