You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tinkerpop.apache.org by dimas-b <gi...@git.apache.org> on 2018/07/27 15:10:42 UTC

[GitHub] tinkerpop pull request #896: TINKERPOP-2005 Reject multiple final responses ...

GitHub user dimas-b opened a pull request:

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

    TINKERPOP-2005 Reject multiple final responses in AbstractEvalOpProcessor

    Add isFinalResponse() getter to ResponseStatusCode
    
    Introduce writeAndFlush methods in Context to allow tracking the final
    response status per request message.
    
    Update AbstractOpProcessor and related classes to write response
    messages through Context methods as opposed to ChannelHandlerContext
    methods.

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

    $ git pull https://github.com/dimas-b/tinkerpop TINKERPOP-2005

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

    https://github.com/apache/tinkerpop/pull/896.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 #896
    
----
commit bd5e86cf6afccafc6f399773b9311dd7b51543d5
Author: Dmitri Bourlatchkov <dm...@...>
Date:   2018-07-27T14:58:40Z

    TINKERPOP-2005 Reject multiple final responses in AbstractEvalOpProcessor
    
    Add isFinalResponse() getter to ResponseStatusCode
    
    Introduce writeAndFlush methods in Context to allow tracking the final
    response status per request message.
    
    Update AbstractOpProcessor and related classes to write response
    messages through Context methods as opposed to ChannelHandlerContext
    methods.

----


---

[GitHub] tinkerpop pull request #896: TINKERPOP-2005 Reject multiple final responses ...

Posted by dimas-b <gi...@git.apache.org>.
Github user dimas-b closed the pull request at:

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


---

[GitHub] tinkerpop issue #896: TINKERPOP-2005 Reject multiple final responses in Abst...

Posted by dimas-b <gi...@git.apache.org>.
Github user dimas-b commented on the issue:

    https://github.com/apache/tinkerpop/pull/896
  
    Alternative PR submitted in #899 


---