[GitHub] tinkerpop issue #820: TINKERPOP-1865 Run Gremlin.Net tests with GraphSON 3.0

2018-03-20 Thread jorgebay
Github user jorgebay commented on the issue: https://github.com/apache/tinkerpop/pull/820 I've rebased this branch to get the fixes for `P.within()` from #817. Also, I've switched to use a lambda (now that we support it :) ) to obtain all the edges for the scenario data, that

[jira] [Commented] (TINKERPOP-1865) Run Gremlin .NET GLV tests with GraphSON 3.0

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16406011#comment-16406011 ] ASF GitHub Bot commented on TINKERPOP-1865: --- Github user jorgebay commented

[GitHub] tinkerpop issue #820: TINKERPOP-1865 Run Gremlin.Net tests with GraphSON 3.0

2018-03-20 Thread FlorianHockmann
Github user FlorianHockmann commented on the issue: https://github.com/apache/tinkerpop/pull/820 > Also, I've switched to use a lambda (now that we support it :) ) to obtain all the edges for the scenario data, That lambda really makes the traversal much easier to read and und

[jira] [Commented] (TINKERPOP-1865) Run Gremlin .NET GLV tests with GraphSON 3.0

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16406231#comment-16406231 ] ASF GitHub Bot commented on TINKERPOP-1865: --- Github user FlorianHockmann com

Re: [DISCUSS] Use error-prone java compiler

2018-03-20 Thread Florian Hockmann
In general, I am +1 on adding a static analysis tool to our tool chain. I just wonder whether a Java compiler is the best choice for us right now or whether other static analysis tools would find most of the problems that such a compiler can find and also work with other languages than Java. SonarQ

[GitHub] tinkerpop issue #820: TINKERPOP-1865 Run Gremlin.Net tests with GraphSON 3.0

2018-03-20 Thread jorgebay
Github user jorgebay commented on the issue: https://github.com/apache/tinkerpop/pull/820 I agree, we should fix `g_V_storeXaX_byXoutEXcreatedX_countX_out_out_storeXaX_byXinEXcreatedX_weight_sumX` outside of this pr. Having the gherkin based tests running on GraphSON3 for Gre

[jira] [Commented] (TINKERPOP-1865) Run Gremlin .NET GLV tests with GraphSON 3.0

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16406297#comment-16406297 ] ASF GitHub Bot commented on TINKERPOP-1865: --- Github user jorgebay commented

[jira] [Updated] (TINKERPOP-1927) Gherkin scenario expects list with duplicates, but receives g:Set

2018-03-20 Thread Florian Hockmann (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1927?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Florian Hockmann updated TINKERPOP-1927: Description: The scenario {{g_V_storeXaX_byXoutEXcreatedX_countX_out_out_storeX

[jira] [Created] (TINKERPOP-1927) Gherkin scenario expects list with duplicates, but receives g:Set

2018-03-20 Thread Florian Hockmann (JIRA)
Florian Hockmann created TINKERPOP-1927: --- Summary: Gherkin scenario expects list with duplicates, but receives g:Set Key: TINKERPOP-1927 URL: https://issues.apache.org/jira/browse/TINKERPOP-1927

[GitHub] tinkerpop issue #820: TINKERPOP-1865 Run Gremlin.Net tests with GraphSON 3.0

2018-03-20 Thread FlorianHockmann
Github user FlorianHockmann commented on the issue: https://github.com/apache/tinkerpop/pull/820 Ok, I created [TINKERPOP-1927](https://issues.apache.org/jira/browse/TINKERPOP-1927) to fix the scenarios. VOTE +1 ---

[jira] [Commented] (TINKERPOP-1865) Run Gremlin .NET GLV tests with GraphSON 3.0

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16406313#comment-16406313 ] ASF GitHub Bot commented on TINKERPOP-1865: --- Github user FlorianHockmann com

[GitHub] tinkerpop issue #813: TINKERPOP-1914 Support construct a GremlinServer insta...

2018-03-20 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/813 ok - thanks. would you mind closing this PR please? i will merge forward the other one to master. ---

[jira] [Commented] (TINKERPOP-1914) Support construct a GremlinServer instance from gremlin executor service

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1914?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16406481#comment-16406481 ] ASF GitHub Bot commented on TINKERPOP-1914: --- Github user spmallette commente

[jira] [Closed] (TINKERPOP-1914) Support construct a GremlinServer instance from gremlin executor service

2018-03-20 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1914?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette closed TINKERPOP-1914. --- Resolution: Done Assignee: stephen mallette Fix Version/s: 3.3.2 >

[GitHub] tinkerpop pull request #821: TINKERPOP-1914 Support construct a GremlinServe...

2018-03-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/821 ---

[jira] [Commented] (TINKERPOP-1914) Support construct a GremlinServer instance from gremlin executor service

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1914?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16406562#comment-16406562 ] ASF GitHub Bot commented on TINKERPOP-1914: --- Github user asfgit closed the p

[GitHub] tinkerpop issue #811: TINKERPOP-1758 Apply RemoteStrategy before all Decorat...

2018-03-20 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/811 Does this handle custom strategies? ---

[jira] [Commented] (TINKERPOP-1758) RemoteStrategy should be before all other DecorationStrategies.

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1758?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16406844#comment-16406844 ] ASF GitHub Bot commented on TINKERPOP-1758: --- Github user robertdale commente

[GitHub] tinkerpop issue #811: TINKERPOP-1758 Apply RemoteStrategy before all Decorat...

2018-03-20 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/811 no - this only works by convention @robertdale - i don't think there's a way to do that short of scanning the classpath somehow for "decorations" and ordering them properly. or perhaps somethin

[jira] [Commented] (TINKERPOP-1758) RemoteStrategy should be before all other DecorationStrategies.

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1758?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16406852#comment-16406852 ] ASF GitHub Bot commented on TINKERPOP-1758: --- Github user spmallette commente

[jira] [Commented] (TINKERPOP-1758) RemoteStrategy should be before all other DecorationStrategies.

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1758?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16406858#comment-16406858 ] ASF GitHub Bot commented on TINKERPOP-1758: --- Github user robertdale commente

[GitHub] tinkerpop issue #811: TINKERPOP-1758 Apply RemoteStrategy before all Decorat...

2018-03-20 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/811 I don't know. Just asking for the record. VOTE +1 ---

[GitHub] tinkerpop issue #811: TINKERPOP-1758 Apply RemoteStrategy before all Decorat...

2018-03-20 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/811 fair enough - i kinda wanted to just fix this problem since it was out there for so long. i'm hesitant to just hardcode the `RemoteStrategy` as "first" - something feels off about doing that. p

[jira] [Commented] (TINKERPOP-1758) RemoteStrategy should be before all other DecorationStrategies.

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1758?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16406862#comment-16406862 ] ASF GitHub Bot commented on TINKERPOP-1758: --- Github user spmallette commente

[GitHub] tinkerpop pull request #813: TINKERPOP-1914 Support construct a GremlinServe...

2018-03-20 Thread javeme
Github user javeme closed the pull request at: https://github.com/apache/tinkerpop/pull/813 ---

[jira] [Commented] (TINKERPOP-1914) Support construct a GremlinServer instance from gremlin executor service

2018-03-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1914?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16407371#comment-16407371 ] ASF GitHub Bot commented on TINKERPOP-1914: --- Github user javeme closed the p