[GitHub] tinkerpop pull request #805: TINKERPOP-1777 Gremlin .max step returns -21474...

2018-03-06 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/805#discussion_r172629315 --- Diff: gremlin-test/features/branch/Union.feature --- @@ -135,7 +135,6 @@ Feature: Step - union() | d[3].l | | d[0].l |

[jira] [Commented] (TINKERPOP-1777) Gremlin .max step returns -2147483648 for empty result sets

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

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388432#comment-16388432 ] stephen mallette commented on TINKERPOP-1896: - when you say, "GremlinGroovy does not work

[jira] [Commented] (TINKERPOP-1885) Various Gremlin.Net documentation updates

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388493#comment-16388493 ] stephen mallette commented on TINKERPOP-1885: - Looks good to me. I just republished the

[DISCUSS] Add cypher-for-gremlin to Index Listing

2018-03-06 Thread Stephen Mallette
It was announced last week that cypher-for-gremlin was available and as of today there was an official release to maven central. I think we can get this added to the index: https://github.com/opencypher/cypher-for-gremlin I'll assume lazy consensus in 72 hours and proceed, unless there are

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-06 Thread Branden Moore (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388449#comment-16388449 ] Branden Moore commented on TINKERPOP-1896: -- I view the DSLs as extensions to the Gremlin

[GitHub] tinkerpop pull request #803: TINKERPOP-1522 Order of select() scopes

2018-03-06 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/803#discussion_r172619337 --- Diff: docs/src/upgrade/release-3.4.x.asciidoc --- @@ -28,3 +28,8 @@ NEED AN IMAGE Please see the

[jira] [Commented] (TINKERPOP-1522) Order of select() scopes

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

[GitHub] tinkerpop issue #803: TINKERPOP-1522 Order of select() scopes

2018-03-06 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/803 VOTE +1 - pending my comment on the upgrade docs ---

[jira] [Closed] (TINKERPOP-1885) Various Gremlin.Net documentation updates

2018-03-06 Thread Florian Hockmann (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Florian Hockmann closed TINKERPOP-1885. --- Resolution: Fixed > Various Gremlin.Net documentation updates >

[GitHub] tinkerpop pull request #812: TINKERPOP-1911 Refactored JavaTranslator

2018-03-06 Thread spmallette
GitHub user spmallette opened a pull request: https://github.com/apache/tinkerpop/pull/812 TINKERPOP-1911 Refactored JavaTranslator Cached methods that were being reflected on every translation which improved performance a bit. Here's how it runs before the change: ```text

[jira] [Commented] (TINKERPOP-1911) Refactor JavaTranslator to cache all reflective calls

2018-03-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388277#comment-16388277 ] ASF GitHub Bot commented on TINKERPOP-1911: --- GitHub user spmallette opened a pull request:

[jira] [Commented] (TINKERPOP-1911) Refactor JavaTranslator to cache all reflective calls

2018-03-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388332#comment-16388332 ] ASF GitHub Bot commented on TINKERPOP-1911: --- Github user BrynCooke commented on the issue:

[GitHub] tinkerpop issue #812: TINKERPOP-1911 Refactored JavaTranslator

2018-03-06 Thread BrynCooke
Github user BrynCooke commented on the issue: https://github.com/apache/tinkerpop/pull/812 +1 (Non binding) ---

[jira] [Commented] (TINKERPOP-1885) Various Gremlin.Net documentation updates

2018-03-06 Thread Florian Hockmann (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388467#comment-16388467 ] Florian Hockmann commented on TINKERPOP-1885: - Ok, I think that I got everything now that

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388469#comment-16388469 ] stephen mallette commented on TINKERPOP-1896: - yep - i agree that what you pose there is a

[jira] [Commented] (TINKERPOP-1777) Gremlin .max step returns -2147483648 for empty result sets

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

[GitHub] tinkerpop pull request #805: TINKERPOP-1777 Gremlin .max step returns -21474...

2018-03-06 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/805#discussion_r172627151 --- Diff: docs/src/upgrade/release-3.4.x.asciidoc --- @@ -28,3 +28,45 @@ NEED AN IMAGE Please see the

[jira] [Closed] (TINKERPOP-1907) Fix failing GLV test for withSack() in .NET

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1907?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette closed TINKERPOP-1907. --- Resolution: Fixed Assignee: stephen mallette I think this was fixed as part

[jira] [Closed] (TINKERPOP-1868) Support inject source step in Gremlin.Net

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1868?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette closed TINKERPOP-1868. --- [~Florian Hockmann] you can just "close" issues once merged - no need to "resolve"

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-06 Thread Branden Moore (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388488#comment-16388488 ] Branden Moore commented on TINKERPOP-1896: -- Right now, my work-around for this ticket is to

[jira] [Commented] (TINKERPOP-1868) Support inject source step in Gremlin.Net

2018-03-06 Thread Florian Hockmann (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1868?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388403#comment-16388403 ] Florian Hockmann commented on TINKERPOP-1868: - Good to know, Jira is a bit confusing

[jira] [Assigned] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette reassigned TINKERPOP-1896: --- Assignee: stephen mallette > gremlin-python lambdas error when working

RE: [Discuss] Expose metadata from Gremlin Server to Clients.

2018-03-06 Thread Ashwini Singh
Create a incident: https://issues.apache.org/jira/browse/TINKERPOP-1913 For this, I am fine with that. Lets take this as part of above work item. For #3, Do we need to take this in this work item as well ? Can someone setup the branch? And we can take it from there. Thanks a lot guys, Ashwini

[GitHub] tinkerpop issue #812: TINKERPOP-1911 Refactored JavaTranslator

2018-03-06 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/812 The local cache is specific to the `GraphTraversalSource` that's passed into the `of()` construction method. As the `GraphTraversalSource` is specific to the instance and is not known until

[jira] [Updated] (TINKERPOP-1913) Expose metadata from Gremlin Server to Clients

2018-03-06 Thread Ashwini Singh (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1913?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ashwini Singh updated TINKERPOP-1913: - Description: To summarize what we have discussed so far:  1. For API using

[jira] [Commented] (TINKERPOP-1896) gremlin-python lambdas error when working on Maps

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388513#comment-16388513 ] stephen mallette commented on TINKERPOP-1896: - ok - well then i'm going to re-focus this

[jira] [Commented] (TINKERPOP-1911) Refactor JavaTranslator to cache all reflective calls

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

[jira] [Updated] (TINKERPOP-1913) Expose metadata from Gremlin Server to Clients

2018-03-06 Thread Ashwini Singh (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1913?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ashwini Singh updated TINKERPOP-1913: - Affects Version/s: 3.3.1 > Expose metadata from Gremlin Server to Clients >

[jira] [Updated] (TINKERPOP-1898) Issue with bindings in strategies and lambdas

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1898?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette updated TINKERPOP-1898: Issue Type: Bug (was: Improvement) > Issue with bindings in strategies and

[jira] [Created] (TINKERPOP-1913) Expose metadata from Gremlin Server to Clients

2018-03-06 Thread Ashwini Singh (JIRA)
Ashwini Singh created TINKERPOP-1913: Summary: Expose metadata from Gremlin Server to Clients Key: TINKERPOP-1913 URL: https://issues.apache.org/jira/browse/TINKERPOP-1913 Project: TinkerPop

[GitHub] tinkerpop issue #812: TINKERPOP-1911 Refactored JavaTranslator

2018-03-06 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/812 What's the difference between the global method cache and the local method cache? ---

[jira] [Commented] (TINKERPOP-1911) Refactor JavaTranslator to cache all reflective calls

2018-03-06 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388529#comment-16388529 ] ASF GitHub Bot commented on TINKERPOP-1911: --- Github user robertdale commented on the issue:

[jira] [Updated] (TINKERPOP-1913) Expose metadata from Gremlin Server to Clients

2018-03-06 Thread Ashwini Singh (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1913?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ashwini Singh updated TINKERPOP-1913: - Description: To summarize what we have discussed so far:  1. For API using

[jira] [Commented] (TINKERPOP-1885) Various Gremlin.Net documentation updates

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388099#comment-16388099 ] stephen mallette commented on TINKERPOP-1885: - I think that the logo is fine - don't think

[jira] [Created] (TINKERPOP-1911) Refactor JavaTranslator to cache all reflective calls

2018-03-06 Thread stephen mallette (JIRA)
stephen mallette created TINKERPOP-1911: --- Summary: Refactor JavaTranslator to cache all reflective calls Key: TINKERPOP-1911 URL: https://issues.apache.org/jira/browse/TINKERPOP-1911 Project:

[jira] [Created] (TINKERPOP-1912) Remove MD5 checksums

2018-03-06 Thread Daniel Kuppitz (JIRA)
Daniel Kuppitz created TINKERPOP-1912: - Summary: Remove MD5 checksums Key: TINKERPOP-1912 URL: https://issues.apache.org/jira/browse/TINKERPOP-1912 Project: TinkerPop Issue Type:

[jira] [Commented] (TINKERPOP-1909) Gremlin.Net does not have complete object model as compared to other client drivers and unable to de-serialize properties for vertex/edge graphSON.

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16387868#comment-16387868 ] stephen mallette commented on TINKERPOP-1909: - Ok - to simplify our discussion, I think we

[jira] [Updated] (TINKERPOP-1912) Remove MD5 checksums

2018-03-06 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1912?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette updated TINKERPOP-1912: Affects Version/s: 3.2.7 > Remove MD5 checksums > > >

[jira] [Updated] (TINKERPOP-1885) Various Gremlin.Net documentation updates

2018-03-06 Thread Florian Hockmann (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Florian Hockmann updated TINKERPOP-1885: Attachment: gremlin-dotnet-logo.png > Various Gremlin.Net documentation updates

[jira] [Updated] (TINKERPOP-1885) Various Gremlin.Net documentation updates

2018-03-06 Thread Florian Hockmann (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Florian Hockmann updated TINKERPOP-1885: Attachment: gremlin-dotnet-logo.png > Various Gremlin.Net documentation updates

[jira] [Commented] (TINKERPOP-1885) Various Gremlin.Net documentation updates

2018-03-06 Thread Florian Hockmann (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388093#comment-16388093 ] Florian Hockmann commented on TINKERPOP-1885: - There seems to be a new .NET logo which is

Re: [Discuss] Expose metadata from Gremlin Server to Clients.

2018-03-06 Thread Stephen Mallette
Your summary looks good to me. I also agree that we can omit 3b from this body of work. Going back to an earlier question that you had: > Can we spilt the change to support for gremlin request/script first and bytecode/traversal separately? I'm a bit concerned that if we don't at least have