[DISCUSS] kotlin-gremlin-ogm index listing

2018-06-25 Thread Stephen Mallette
An OGM for Kotlin was just announced on gremlin-users: https://groups.google.com/d/msg/gremlin-users/vJFQJPvq4T8/x978GmYQCAAJ https://github.com/pm-dev/kotlin-gremlin-ogm https://www.excusethedisruption.com/introducing-kotlin-gremlin-ogm/ It looks like it meets the requirements for an index

[jira] [Commented] (TINKERPOP-1365) Log the seed used to initialize Random in tests

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

[jira] [Commented] (TINKERPOP-1989) Preserve order that plugins are applied in Gremlin Console

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

[GitHub] tinkerpop pull request #881: TINKERPOP-1365 Refactored use of Random in test...

2018-06-25 Thread dkuppitz
Github user dkuppitz commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/881#discussion_r197826190 --- Diff: gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/TestHelper.java --- @@ -51,11 +52,19 @@ * @author Stephen Mallette

[jira] [Commented] (TINKERPOP-1365) Log the seed used to initialize Random in tests

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

[GitHub] tinkerpop issue #880: TINKERPOP-1989 Enforce order of plugin load in Gremlin...

2018-06-25 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/880 VOTE: +1 ---

[GitHub] tinkerpop pull request #881: TINKERPOP-1365 Refactored use of Random in test...

2018-06-25 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/881#discussion_r197827356 --- Diff: gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/TestHelper.java --- @@ -51,11 +52,19 @@ * @author Stephen Mallette

Re: CustomId serialization

2018-06-25 Thread Stephen Mallette
I would think that you could write your own custom deserializer if you needed to. That error doesn't give me any hints as to what might be wrong exactly. I can't think of why that wouldn't work, but even with a little refresh by looking at the code just now, my memory on GraphSON 1.0 is fuzzy.

Re: CustomId serialization

2018-06-25 Thread pieter gmail
Hi, Just manage to get it to work, but not really sure whats going on. So Sqlg's RecordId itself consist of a SchemaTable and a Long. Both RecordId and SchemaTable has serialization code. The part I don't quite get is that serializeWithType and deserialize is not symmetrical. Here is

[GitHub] tinkerpop issue #885: TINKERPOP-1978 Gremlin.Net: Add check for connection s...

2018-06-25 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/885 VOTE: +1 ---

[jira] [Commented] (TINKERPOP-1978) Check for Websocket connection state when retrieved from Connection Pool missing

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

[jira] [Commented] (TINKERPOP-1978) Check for Websocket connection state when retrieved from Connection Pool missing

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

[jira] [Commented] (TINKERPOP-1864) Gremlin Python tests for GraphSON 2.0 and 3.0

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

[GitHub] tinkerpop issue #883: TINKERPOP-1864 Run python glv tests on GraphSON 2.0 an...

2018-06-25 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/883 VOTE: +1 ... but maybe make the configuration mandatory and throw an exception if ` world.config.user_data["graphson"]` was not set. ---

[jira] [Commented] (TINKERPOP-1985) Update position on bulk loading

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

Re: CustomId serialization

2018-06-25 Thread Stephen Mallette
I think - "think" being the key word - that Jackson parses that CLASS to determine the deserializer to use and then hands your deserializer the contents of the rest of the JSON (which is all the deserializers needs once the right one is chosen). On Mon, Jun 25, 2018 at 8:08 AM pieter gmail

[GitHub] tinkerpop issue #884: TINKERPOP-1985 Changing position on bulk import/export

2018-06-25 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/884 VOTE: +1 ---

[jira] [Commented] (TINKERPOP-1864) Gremlin Python tests for GraphSON 2.0 and 3.0

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

Re: CustomId serialization

2018-06-25 Thread pieter gmail
Ok, ah well I briefly searched for "@class" magic and did not find it. I find Jackson's docs surprisingly bad. Anyway no matter, it works for now. Thanks Pieter On 25/06/2018 18:48, Stephen Mallette wrote: I think - "think" being the key word - that Jackson parses that CLASS to determine the

[GitHub] tinkerpop pull request #885: TINKERPOP-1978 Gremlin.Net: Add check for conne...

2018-06-25 Thread dkuppitz
Github user dkuppitz commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/885#discussion_r197851424 --- Diff: gremlin-dotnet/src/Gremlin.Net/Driver/ConnectionPool.cs --- @@ -45,19 +44,28 @@ public ConnectionPool(ConnectionFactory connectionFactory)

[GitHub] tinkerpop issue #883: TINKERPOP-1864 Run python glv tests on GraphSON 2.0 an...

2018-06-25 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/883 That's fair - changed now so that you get this if "graphson" isn't set: ```text [exec] Error: Hook 'prepare_traversal_source' from

[jira] [Commented] (TINKERPOP-1978) Check for Websocket connection state when retrieved from Connection Pool missing

2018-06-25 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16522475#comment-16522475 ] ASF GitHub Bot commented on TINKERPOP-1978: --- Github user FlorianHockmann commented on a

[GitHub] tinkerpop pull request #885: TINKERPOP-1978 Gremlin.Net: Add check for conne...

2018-06-25 Thread FlorianHockmann
Github user FlorianHockmann commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/885#discussion_r197855449 --- Diff: gremlin-dotnet/src/Gremlin.Net/Driver/ConnectionPool.cs --- @@ -45,19 +44,28 @@ public ConnectionPool(ConnectionFactory

[GitHub] tinkerpop issue #876: TINKERPOP-967 Support nested-repeat() structures

2018-06-25 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/876 I agree it should be treated differently. Since `as()` labeling has an impact on the path history, it really shouldn't be used for this case. ---

[jira] [Commented] (TINKERPOP-967) Support nested-repeat() structures

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

[GitHub] tinkerpop issue #876: TINKERPOP-967 Support nested-repeat() structures

2018-06-25 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/876 > perhaps there is a reason to unite the two concepts. can't really think of a reason to "unite the two concepts" - I think that we want to keep it as `repeat(‘a’,…) !=

[jira] [Commented] (TINKERPOP-967) Support nested-repeat() structures

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

RE: CustomId serialization

2018-06-25 Thread pieter gmail
Hi, I am trying to upgrade Sqlg to 3.3.3 from 3.3.1. The only tests that are failing are the io tests for graphson V1. I see CustomId has a CustomIdJacksonSerializerV1d0 but not a deserializer. Looks like Jackson is using reflection to instantiate the CustomId and set its cluster and