Let JavaTranslator.translateObject() throw clearer ISE

Wrap the full original exception, because its message might be enough to pinpoint the issue.
xiazcy · Dec 6, 2023 · 500252d · 500252d
1 parent 1e0cfae
commit 500252d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
@@ -23,7 +23,7 @@ image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima
 [[release-3-6-7]]
 === TinkerPop 3.6.7 (NOT OFFICIALLY RELEASED YET)
 
-
+* Improved error message from `JavaTranslator` by including exception source.
 
 [[release-3-6-6]]
 === TinkerPop 3.6.6 (November 20, 2023)

diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/jsr223/JavaTranslator.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/jsr223/JavaTranslator.java
@@ -118,7 +118,7 @@ else if (object instanceof Bytecode) {
                 }
                 return traversal;
             } catch (final Throwable e) {
-                throw new IllegalStateException(e.getMessage());
+                throw new IllegalStateException(e.getMessage(), e);
             }
         } else if (object instanceof TraversalStrategyProxy) {
             final Map<String, Object> map = new HashMap<>();