GitHub user spmallette opened a pull request:

    https://github.com/apache/incubator-tinkerpop/pull/294

    TINKERPOP-1267 Added option for "none" on remote timeouts.

    Added some tests to validate the new timeout setting and did some manual 
tests:
    
    ```text
    gremlin> :remote connect tinkerpop.server conf/remote.yaml
    ==>Connected - localhost/127.0.0.1:8182
    gremlin> :remote config timeout max
    ==>Set remote timeout to 2147483647ms
    gremlin> :remote config timeout none
    ==>Remote timeout is disable
    gremlin> :remote config timeout 10000
    ==>Set remote timeout to 10000ms
    gremlin> :> Thread.sleep(9000)
    ==>null
    gremlin> :> Thread.sleep(11000)
    Request timed out while processing - increase the timeout with the :remote 
command
    Display stack trace? [yN] n
    ```
    
    Successful run of `mvn clean install && mvn verify -pl gremlin-console 
-DskipIntegrationTests=false`
    
    VOTE +1


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

    $ git pull https://github.com/apache/incubator-tinkerpop TINKERPOP-1267

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

    https://github.com/apache/incubator-tinkerpop/pull/294.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 #294
    
----

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to