[jira] [Commented] (TINKERPOP-1490) Provider a Future based Traversal.async(Function<Traversal,V>) terminal step

2016-10-06 Thread Jorge Bay (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551199#comment-15551199 ] Jorge Bay commented on TINKERPOP-1490: -- {{async}} is a reserved keyword in many languages: - In

[GitHub] tinkerpop issue #450: TINKERPOP-1489 Javascript GLV

2016-10-06 Thread jorgebay
Github user jorgebay commented on the issue: https://github.com/apache/tinkerpop/pull/450 1: I mentioned the difficulty of delivering a `RemoteConnection` implementation inside the TinkerPop repository above. 2: I commented in TINKERPOP-1490 that `async` is a reserved keyword, in

[jira] [Commented] (TINKERPOP-1489) Provide a Javascript Gremlin Language Variant

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551349#comment-15551349 ] ASF GitHub Bot commented on TINKERPOP-1489: --- Github user jorgebay commented on the issue:

[jira] [Commented] (TINKERPOP-1489) Provide a Javascript Gremlin Language Variant

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551606#comment-15551606 ] ASF GitHub Bot commented on TINKERPOP-1489: --- Github user spmallette commented on a diff in

[GitHub] tinkerpop issue #449: TINKERPOP-1488: Make LazyBarrierStrategy part of the d...

2016-10-06 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/449 This is a breaking change in the sense as the package name changed. I doubt it should break most users, but a breaking change nonetheless. I think it would have been better to deprecate it

[jira] [Commented] (TINKERPOP-1489) Provide a Javascript Gremlin Language Variant

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551631#comment-15551631 ] ASF GitHub Bot commented on TINKERPOP-1489: --- Github user jorgebay commented on a diff in the

[jira] [Commented] (TINKERPOP-1488) Make LazyBarrierStrategy part of the default TraversalStrategies

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551622#comment-15551622 ] ASF GitHub Bot commented on TINKERPOP-1488: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop pull request #450: TINKERPOP-1489 Javascript GLV

2016-10-06 Thread jorgebay
Github user jorgebay commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/450#discussion_r82163673 --- Diff: gremlin-javascript/pom.xml --- @@ -0,0 +1,132 @@ + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] tinkerpop pull request #450: TINKERPOP-1489 Javascript GLV

2016-10-06 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/450#discussion_r82167244 --- Diff: gremlin-javascript/pom.xml --- @@ -0,0 +1,132 @@ + + +http://maven.apache.org/POM/4.0.0; +

[jira] [Commented] (TINKERPOP-1158) gremlin.sh -v emits log4j initialization errors

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15553308#comment-15553308 ] ASF GitHub Bot commented on TINKERPOP-1158: --- Github user asfgit closed the pull request at:

[jira] [Closed] (TINKERPOP-1158) gremlin.sh -v emits log4j initialization errors

2016-10-06 Thread Daniel Kuppitz (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1158?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Kuppitz closed TINKERPOP-1158. - Resolution: Fixed Fix Version/s: 3.2.3 3.1.5 > gremlin.sh

[GitHub] tinkerpop issue #444: TINKERPOP-1158 gremlin.sh -v emits log4j initializatio...

2016-10-06 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/444 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

[jira] [Closed] (TINKERPOP-1488) Make LazyBarrierStrategy part of the default TraversalStrategies

2016-10-06 Thread Marko A. Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marko A. Rodriguez closed TINKERPOP-1488. - Resolution: Fixed Fix Version/s: 3.2.3 > Make LazyBarrierStrategy part

[jira] [Commented] (TINKERPOP-1458) Gremlin Server doesn't return confirmation upon Traversal OpProcessor "close" op

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552585#comment-15552585 ] ASF GitHub Bot commented on TINKERPOP-1458: --- Github user davebshow commented on a diff in

[GitHub] tinkerpop issue #448: Python glv graphson update

2016-10-06 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/448 This looks good. However, can you please rebase with `master/` as there have been some changes to `graphson.py`. Also, if there is anything that needs to be added to `gremlin-variants.asciidoc`

[jira] [Commented] (TINKERPOP-1158) gremlin.sh -v emits log4j initialization errors

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552763#comment-15552763 ] ASF GitHub Bot commented on TINKERPOP-1158: --- Github user okram commented on the issue:

[jira] [Commented] (TINKERPOP-1458) Gremlin Server doesn't return confirmation upon Traversal OpProcessor "close" op

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552628#comment-15552628 ] ASF GitHub Bot commented on TINKERPOP-1458: --- Github user spmallette commented on a diff in

[GitHub] tinkerpop issue #449: TINKERPOP-1488: Make LazyBarrierStrategy part of the d...

2016-10-06 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/449 All tests pass with `docker/build.sh -t -i` 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

[GitHub] tinkerpop issue #448: Python glv graphson update

2016-10-06 Thread aholmberg
Github user aholmberg commented on the issue: https://github.com/apache/tinkerpop/pull/448 I'm ambivalent on that. I just noticed unittest was present, and the bare assertions were not very informative. I can take that commit out if you think it's not worth it now. --- If your

[jira] [Closed] (TINKERPOP-1436) GraphSON 2.0 documentation on types

2016-10-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1436?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette closed TINKERPOP-1436. --- Resolution: Duplicate Assignee: stephen mallette > GraphSON 2.0

[jira] [Commented] (TINKERPOP-1488) Make LazyBarrierStrategy part of the default TraversalStrategies

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552772#comment-15552772 ] ASF GitHub Bot commented on TINKERPOP-1488: --- Github user okram commented on the issue:

[GitHub] tinkerpop issue #449: TINKERPOP-1488: Make LazyBarrierStrategy part of the d...

2016-10-06 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/449 @spmallette -- good call. I added back `finalization.LazyBarrierStrategy` and @Deprecated it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[jira] [Commented] (TINKERPOP-1455) Provide String-based withStrategy()/withoutStrategy() for language variant usage

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552610#comment-15552610 ] ASF GitHub Bot commented on TINKERPOP-1455: --- Github user asfgit closed the pull request at:

[GitHub] tinkerpop pull request #447: TINKERPOP-1455: Provide String-based withStrate...

2016-10-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/447 --- 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

[GitHub] tinkerpop pull request #451: TINKERPOP-1458 Gremlin Server doesn't return co...

2016-10-06 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/451#discussion_r82243790 --- Diff: gremlin-driver/src/test/java/org/apache/tinkerpop/gremlin/driver/remote/DriverRemoteTraversalSideEffectsTest.java --- @@ -0,0 +1,116 @@

[GitHub] tinkerpop pull request #451: TINKERPOP-1458 Gremlin Server doesn't return co...

2016-10-06 Thread davebshow
Github user davebshow commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/451#discussion_r82239737 --- Diff: gremlin-driver/src/test/java/org/apache/tinkerpop/gremlin/driver/remote/DriverRemoteTraversalSideEffectsTest.java --- @@ -0,0 +1,116 @@

[GitHub] tinkerpop issue #440: TINKERPOP-1044: Gremlin server REST endpoint - Add Exc...

2016-10-06 Thread vrkrishn
Github user vrkrishn commented on the issue: https://github.com/apache/tinkerpop/pull/440 Would it be possible to push this change then and add the other functionality as a different ticket? --- If your project is set up for it, you can reply to this email and have your reply appear

[jira] [Commented] (TINKERPOP-1044) ResponseMessage should contain server-side exception name.

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1044?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552454#comment-15552454 ] ASF GitHub Bot commented on TINKERPOP-1044: --- Github user vrkrishn commented on the issue:

[jira] [Commented] (TINKERPOP-1433) Add steps to dev docs to help committers get their keys in order

2016-10-06 Thread Ted Wilmes (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1433?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552512#comment-15552512 ] Ted Wilmes commented on TINKERPOP-1433: --- Yes, I need to do this and then incorporate a few other

[GitHub] tinkerpop issue #449: TINKERPOP-1488: Make LazyBarrierStrategy part of the d...

2016-10-06 Thread twilmes
Github user twilmes commented on the issue: https://github.com/apache/tinkerpop/pull/449 The code and tests look good to me. `mvn clean install` also ran without issue. VOTE: +1 --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] tinkerpop pull request #451: TINKERPOP-1458 Gremlin Server doesn't return co...

2016-10-06 Thread davebshow
Github user davebshow commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/451#discussion_r82311244 --- Diff: gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/remote/DriverRemoteTraversalSideEffects.java --- @@ -39,12 +40,14 @@

[jira] [Commented] (TINKERPOP-1458) Gremlin Server doesn't return confirmation upon Traversal OpProcessor "close" op

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552165#comment-15552165 ] ASF GitHub Bot commented on TINKERPOP-1458: --- Github user spmallette commented on a diff in

[GitHub] tinkerpop pull request #451: TINKERPOP-1458 Gremlin Server doesn't return co...

2016-10-06 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/451#discussion_r82211073 --- Diff: gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/remote/DriverRemoteTraversalSideEffects.java --- @@ -54,22 +57,30 @@ public

[jira] [Commented] (TINKERPOP-1458) Gremlin Server doesn't return confirmation upon Traversal OpProcessor "close" op

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552175#comment-15552175 ] ASF GitHub Bot commented on TINKERPOP-1458: --- Github user spmallette commented on a diff in

[GitHub] tinkerpop pull request #451: TINKERPOP-1458 Gremlin Server doesn't return co...

2016-10-06 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/451#discussion_r82210649 --- Diff: gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/remote/DriverRemoteTraversalSideEffects.java --- @@ -89,9 +100,8 @@ public

[jira] [Commented] (TINKERPOP-1458) Gremlin Server doesn't return confirmation upon Traversal OpProcessor "close" op

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552160#comment-15552160 ] ASF GitHub Bot commented on TINKERPOP-1458: --- Github user spmallette commented on a diff in

[GitHub] tinkerpop issue #440: TINKERPOP-1044: Gremlin server REST endpoint - Add Exc...

2016-10-06 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/440 We intend to freeze the code base for release of 3.2.3 (and 3.1.5) in a couple of days. If you'd like to see this merged in time for 3.2.3, we'd need to have your changes pretty soon for

[GitHub] tinkerpop pull request #451: TINKERPOP-1458 Gremlin Server doesn't return co...

2016-10-06 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/451#discussion_r82210120 --- Diff: gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/remote/DriverRemoteTraversalSideEffects.java --- @@ -89,9 +100,8 @@ public

[GitHub] tinkerpop pull request #451: TINKERPOP-1458 Gremlin Server doesn't return co...

2016-10-06 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/451#discussion_r82212090 --- Diff: gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerIntegrateTest.java --- @@ -814,18 +829,88 @@ public void

[jira] [Commented] (TINKERPOP-1489) Provide a Javascript Gremlin Language Variant

2016-10-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551680#comment-15551680 ] ASF GitHub Bot commented on TINKERPOP-1489: --- Github user jorgebay commented on a diff in the

[GitHub] tinkerpop pull request #450: TINKERPOP-1489 Javascript GLV

2016-10-06 Thread jorgebay
Github user jorgebay commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/450#discussion_r82168010 --- Diff: gremlin-javascript/pom.xml --- @@ -0,0 +1,132 @@ + + +http://maven.apache.org/POM/4.0.0; +

[jira] [Commented] (TINKERPOP-1433) Add steps to dev docs to help committers get their keys in order

2016-10-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1433?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552322#comment-15552322 ] stephen mallette commented on TINKERPOP-1433: - Didn't notice this issue before today -