Github user tolbertam commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/605#discussion_r113846423
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/remote/RemoteConnection.java
---
@@ -67,7 +67,11 @@
// default implementation for backward compatibility to 3.2.4 -
this method will probably just become
// the new submit() in 3.3.x when the deprecation is removed
final CompletableFuture<RemoteTraversal<?, E>> promise = new
CompletableFuture<>();
- promise.complete(submit(bytecode));
+ try {
+ promise.complete(submit(bytecode));
+ } catch (Exception t) {
--- End diff --
Chose to limit to `Exception` types instead of `Throwable` since
`RuntimeException`/`Error` are probably acceptable to be thrown on calling
thread since they are unexpected and likely not caused by the traversal failing.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
