GitHub user spmallette opened a pull request:

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

    TINKERPOP-1714 GremlinExecutor checks for timeout from time script submitted

    https://issues.apache.org/jira/browse/TINKERPOP-1714
    
    Prior to this change timeouts were considered from the time the script 
started evaluation which probably isn't in line with user expectations but also 
made it hard to clear the job queue on an overloaded server. This change also 
has the benefit of simplifying the code as well as reducing the number of jobs 
submitted to the various thread pools.
    
    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-1714

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

    https://github.com/apache/tinkerpop/pull/658.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 #658
    
----
commit ac19d5dec659a1a8f399d5a11a4812774eebeca7
Author: Stephen Mallette <[email protected]>
Date:   2017-07-06T15:49:21Z

    TINKERPOP-1714 GremlinExecutor checks for timeout from time script submitted
    
    Prior to this change timeouts were considered from the time the script 
started evaluation which probably isn't in line with user expectations but also 
made it hard to clear the job queue on an overloaded server.

----


---
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.
---

Reply via email to