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

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

GitHub user spmallette opened a pull request:

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

    TinkerPop-1044 master

    https://issues.apache.org/jira/browse/TINKERPOP-1044
    
    Companion pull request for #598 since there were some conflicts that needed 
resolution that were somewhat non-trivial.
    
    All tests pass with `docker/build.sh -t -n -i`
    
    VOTE +1

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/tinkerpop TINKERPOP-1044-master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tinkerpop/pull/599.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #599
    
----
commit b29bab0acada54aa525184935d96b41f9cf0854b
Author: Stephen Mallette <[email protected]>
Date:   2017-04-06T21:55:39Z

    TINKERPOP-1044 Additional error data to Gremlin Server responses
    
    Applies to all protocols: http, nio, and websocket. Added stack trace and 
exception hierarchy. Deprecated the "Exception-Class" in the http protocol.

commit c33069cecd227dff664a02330a8462efce32e121
Author: Stephen Mallette <[email protected]>
Date:   2017-04-06T22:10:08Z

    TINKERPOP-1044 Added documentation for additional error info in 
ResponseMessage

commit 92eceb4e620409be2c9ee810cc4132b7ae2dafc7
Author: Stephen Mallette <[email protected]>
Date:   2017-04-07T12:22:31Z

    Merge branch 'TINKERPOP-1044' into TINKERPOP-1044-master
    
    Conflicts:
        
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/AbstractEvalOpProcessor.java
        
gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerHttpIntegrateTest.java

----


> ResponseMessage should contain server-side exception name.
> ----------------------------------------------------------
>
>                 Key: TINKERPOP-1044
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1044
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: server
>    Affects Versions: 3.1.0-incubating
>            Reporter: Bob Briody
>            Assignee: stephen mallette
>            Priority: Minor
>
> When an exception occurs during execution by gremlin server, the 
> ResponseMessage currently contains the Exception message, but it would also 
> be helpful to include the Exception class name.



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

Reply via email to