[ 
https://issues.apache.org/jira/browse/TINKERPOP-1675?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16024754#comment-16024754
 ] 

ASF GitHub Bot commented on TINKERPOP-1675:
-------------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/tinkerpop/pull/609


> RemoteStep#processNextStart() throws CompletionException instead of 
> underlying exception
> ----------------------------------------------------------------------------------------
>
>                 Key: TINKERPOP-1675
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1675
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: process
>    Affects Versions: 3.2.4
>            Reporter: Andy Tolbert
>
> There is a slight behavior change in TinkerPop 3.2.4 as it pertains to an 
> exception thrown when processing a traversal on a remote traversal source.
> In 3.2.3, if an unchecked exception is thrown, it is propagated directly to 
> the user.
> However, in 3.2.4, since RemoteStep#processNextStart() now calls 
> RemoteStep#promise() and joins on the returning futures result, if the future 
> fails a {{CompletionException}} is thrown wrapping the cause, instead of the 
> cause itself.
> For consistency, if that exception is an unchecked exception, it should be 
> thrown directly, otherwise it should be wrapped in an 
> {{IllegalStateException}} as it did before.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to