You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tinkerpop.apache.org by "Stephen Mallette (Jira)" <ji...@apache.org> on 2021/04/16 10:35:00 UTC

[jira] [Closed] (TINKERPOP-2245) Consolidate the executor for bytecode & string based client

     [ https://issues.apache.org/jira/browse/TINKERPOP-2245?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stephen Mallette closed TINKERPOP-2245.
---------------------------------------
    Fix Version/s: 3.5.0
       Resolution: Done

while we still need to better account for a multi-threaded transaction approach to sessions so that they don't block the threads in the gremlinPool the work as it currently is will still be incredibly useful with remote transactions. we can open more specific issues as we think of them, but this is a good base foundation to start from.

> Consolidate the executor for bytecode & string based client
> -----------------------------------------------------------
>
>                 Key: TINKERPOP-2245
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2245
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: server
>    Affects Versions: 3.4.2
>            Reporter: Divij Vaidya
>            Assignee: Stephen Mallette
>            Priority: Minor
>             Fix For: 3.5.0
>
>
> We have two code paths in the server which perform (more or less) the same functions. One is the executor for string based queries and other is the executor for bytecode. This code can be refactored together so that the logic to handle timeout, handle exception during execution, handle exception before execution and others can consolidated. 
> [https://github.com/apache/tinkerpop/blob/master/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/GremlinExecutor.java#L246]
> and
> [https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/traversal/TraversalOpProcessor.java#L333]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)