You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tinkerpop.apache.org by spmallette <gi...@git.apache.org> on 2017/04/03 11:35:47 UTC

[GitHub] tinkerpop pull request #591: TINKERPOP-1663 Validation for maximum number of...

GitHub user spmallette opened a pull request:

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

    TINKERPOP-1663 Validation for maximum number of parameters on a request

    https://issues.apache.org/jira/browse/TINKERPOP-1663
    
    Provides a configuration option to validate the number of parameters that gremlin server will accept on a request for script evaluation.
    
    Ran integration tests for gremlin-server and console.
    
    vOTE +1

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

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

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

    https://github.com/apache/tinkerpop/pull/591.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 #591
    
----
commit a4c652ae3c071c15c5a57524a797707f5ea318a4
Author: Stephen Mallette <sp...@genoprime.com>
Date:   2017-04-03T11:04:42Z

    TINKERPOP-1663 Validation for maximum number of parameters on a request
    
    The default is set to 16 and there is a configuration option to allow it to be changed.

----


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

[GitHub] tinkerpop pull request #591: TINKERPOP-1663 Validation for maximum number of...

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

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


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

[GitHub] tinkerpop issue #591: TINKERPOP-1663 Validation for maximum number of parame...

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

    https://github.com/apache/tinkerpop/pull/591
  
     VOTE: +1


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

[GitHub] tinkerpop issue #591: TINKERPOP-1663 Validation for maximum number of parame...

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

    https://github.com/apache/tinkerpop/pull/591
  
    LGTM. Also ran server/console integration. Docs are clear.
    
    VOTE +1


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