[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user danielcweber commented on the issue: https://github.com/apache/tinkerpop/pull/842 Thanks for accepting the contribution! For what it's worth, I am developing a [type-safe ORM](https://github.com/ExRam/ExRam.Gremlinq) (abusing the 'R' in ORM a bit ;-) on top of

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user danielcweber commented on the issue: https://github.com/apache/tinkerpop/pull/842 Alright, thanks! ---

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578733#comment-16578733 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user danielcweber commented on the

[GitHub] tinkerpop issue #889: Tinkerpop 1977 - Sasl Authentication

Github user mattallenuk commented on the issue: https://github.com/apache/tinkerpop/pull/889 @jorgebay Yes that is a great idea have implemented and made the changes requested. Have also updated the tests to use the PlainTextSaslAuthenticator class. ---

[jira] [Commented] (TINKERPOP-2024) Gremlin Server Application archetype should connect via withRemote

[ https://issues.apache.org/jira/browse/TINKERPOP-2024?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578705#comment-16578705 ] ASF GitHub Bot commented on TINKERPOP-2024: --- Github user FlorianHockmann commented on the

[GitHub] tinkerpop issue #907: Fix Sack Split.

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/907 @RainMark did you have a chance to consider the previous comment from @robertdale ? While I appreciate that you have a solution to a problem you are seeing, I share his concern about

[jira] [Commented] (TINKERPOP-1878) Sparql to Gremlin Compiler

[ https://issues.apache.org/jira/browse/TINKERPOP-1878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578631#comment-16578631 ] ASF GitHub Bot commented on TINKERPOP-1878: --- Github user spmallette commented on a diff in

[GitHub] tinkerpop pull request #902: TINKERPOP-1878 sparql-gremlin

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/902#discussion_r209688434 --- Diff: sparql-gremlin/src/main/java/org/apache/tinkerpop/gremlin/sparql/process/traversal/dsl/sparql/SparqlTraversalSource.java --- @@ -0,0 +1,155

[GitHub] tinkerpop pull request #842: TINKERPOP-1945

Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/842 ---

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578700#comment-16578700 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user asfgit closed the pull request at:

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578656#comment-16578656 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user danielcweber commented on the

[jira] [Closed] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette closed TINKERPOP-1945. --- Resolution: Done Assignee: Florian Hockmann Fix Version/s: 3.2.10

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578691#comment-16578691 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user danielcweber commented on the

[GitHub] tinkerpop pull request #912: TINKERPOP-2023 SSL Enhancements

GitHub user robertdale opened a pull request: https://github.com/apache/tinkerpop/pull/912 TINKERPOP-2023 SSL Enhancements https://issues.apache.org/jira/browse/TINKERPOP-2023 Gremlin Server no longer supports automatically creating self-signed certificates.

[jira] [Commented] (TINKERPOP-2023) Gremlin Server should not create self-signed certs

[ https://issues.apache.org/jira/browse/TINKERPOP-2023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578828#comment-16578828 ] ASF GitHub Bot commented on TINKERPOP-2023: --- GitHub user robertdale opened a pull request:

Re: [DISCUSS] Options for withRemote()

Why does it have to be a graph at all? Maybe getting a traversal from a graph is the wrong concept and that's why it perplexes you so. We're connecting to a server that can contain one or more graphs. The traversal should probably come off the connection or be standalone. Maybe the traversal

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578696#comment-16578696 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/842 nice @FlorianHockmann - i will manually close this PR now @danielcweber hope we get to see some more pull requests from you in the future. really nicely done - i bragged about your

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/842 neat - I'd be happy to suggest adding it to our provider index on the tinkerpop home page assuming it meets the listing policy: http://tinkerpop.apache.org/policy.html please

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578668#comment-16578668 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop issue #902: TINKERPOP-1878 sparql-gremlin

Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/902 VOTE: +1 ---

[jira] [Commented] (TINKERPOP-1878) Sparql to Gremlin Compiler

[ https://issues.apache.org/jira/browse/TINKERPOP-1878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578686#comment-16578686 ] ASF GitHub Bot commented on TINKERPOP-1878: --- Github user dkuppitz commented on the issue:

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578687#comment-16578687 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user FlorianHockmann commented on the

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user danielcweber commented on the issue: https://github.com/apache/tinkerpop/pull/842 Thanks a lot for the twitter mention! In a completely unrelated issue, is anybody of your team in contact with the graph implementors over at CosmosDB? I have no luck in getting [this

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578716#comment-16578716 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user danielcweber commented on the

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user danielcweber commented on the issue: https://github.com/apache/tinkerpop/pull/842 I guess it could qualify for the listing with some minor tweaks. It doesn't have a release yet (things get deliberately broken by me all the time, so it's at most in some version 0.x, if

[GitHub] tinkerpop issue #911: TINKERPOP-2024 Make server archetype use remote traver...

Github user FlorianHockmann commented on the issue: https://github.com/apache/tinkerpop/pull/911 Thanks for implementing this so quickly, @spmallette LGTM, VOTE +1 ---

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578727#comment-16578727 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/842 I'm not personally in touch with anyone over there really. I usually just ping folks over there on twitter if I need to convey/ask something - try Luis Bosquez:

[GitHub] tinkerpop pull request #902: TINKERPOP-1878 sparql-gremlin

Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/902 ---

[jira] [Commented] (TINKERPOP-1878) Sparql to Gremlin Compiler

[ https://issues.apache.org/jira/browse/TINKERPOP-1878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578977#comment-16578977 ] ASF GitHub Bot commented on TINKERPOP-1878: --- Github user asfgit closed the pull request at:

Re: [DISCUSS] Options for withRemote()

Yes - I did consider killing instantiation through a Graph except that: 1. It felt like a major major departure from what we have been doing and if users have been using things like GraphFactory to instantiate graphs then they might continue to expect that support (currently supported by

[jira] [Closed] (TINKERPOP-1878) Sparql to Gremlin Compiler

[ https://issues.apache.org/jira/browse/TINKERPOP-1878?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette closed TINKERPOP-1878. --- Resolution: Done Fix Version/s: 3.4.0 > Sparql to Gremlin Compiler >

[GitHub] tinkerpop issue #898: gremlin-javascript: Typescript typings definition

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/898 That was going to be my suggestion too - I think that it can be auto-generated then it should be. ---

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/842 I'm working on merging this now. ---

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578359#comment-16578359 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user spmallette commented on the issue:

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16577960#comment-16577960 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user danielcweber commented on the

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user danielcweber commented on the issue: https://github.com/apache/tinkerpop/pull/842 Revised based on the comments, rebased onto the current tp32 branch. ---

[GitHub] tinkerpop issue #898: gremlin-javascript: Typescript typings definition

Github user jorgebay commented on the issue: https://github.com/apache/tinkerpop/pull/898 Nice @daem0ndev! I think its a good idea to provide typing definitions inside the package. [Gremlin Variant

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user FlorianHockmann commented on the issue: https://github.com/apache/tinkerpop/pull/842 Great, thanks for making those changes. My VOTE +1 from nearly 3 months ago still holds with these recent changes. ---

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578175#comment-16578175 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user FlorianHockmann commented on the

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578251#comment-16578251 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user jorgebay commented on a diff in

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user jorgebay commented on the issue: https://github.com/apache/tinkerpop/pull/842 LGTM! VOTE: +1. Thanks @danielcweber for the contribution! Note to committers: I think it would be better to squash the commits ahead of the merge. ---

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578273#comment-16578273 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user jorgebay commented on a diff in

[GitHub] tinkerpop pull request #842: TINKERPOP-1945

Github user jorgebay commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/842#discussion_r209598480 --- Diff: gremlin-dotnet/src/Gremlin.Net/Gremlin.Net.csproj --- @@ -48,7 +48,7 @@ Please see the reference documentation at Apache TinkerPop for more

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578264#comment-16578264 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user FlorianHockmann commented on a

[GitHub] tinkerpop pull request #842: TINKERPOP-1945

Github user FlorianHockmann commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/842#discussion_r209602159 --- Diff: gremlin-dotnet/src/Gremlin.Net/Gremlin.Net.csproj --- @@ -48,7 +48,7 @@ Please see the reference documentation at Apache TinkerPop for

[GitHub] tinkerpop pull request #842: TINKERPOP-1945

Github user jorgebay commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/842#discussion_r209605835 --- Diff: gremlin-dotnet/src/Gremlin.Net/Gremlin.Net.csproj --- @@ -48,7 +48,7 @@ Please see the reference documentation at Apache TinkerPop for more

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578276#comment-16578276 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user jorgebay commented on the issue:

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578286#comment-16578286 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user FlorianHockmann commented on a

[GitHub] tinkerpop pull request #842: TINKERPOP-1945

Github user FlorianHockmann commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/842#discussion_r209610435 --- Diff: gremlin-dotnet/src/Gremlin.Net/Gremlin.Net.csproj --- @@ -48,7 +48,7 @@ Please see the reference documentation at Apache TinkerPop for

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578438#comment-16578438 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/842 I got the merge done on tp32, but when I merged to tp33 .NET wouldn't build. I didn't immediately recognize the problem, so perhaps it's best that @jorgebay or @FlorianHockmann pick it up

[jira] [Commented] (TINKERPOP-1774) Gremlin .NET: Support min and max sizes in Connection pool

[ https://issues.apache.org/jira/browse/TINKERPOP-1774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578443#comment-16578443 ] ASF GitHub Bot commented on TINKERPOP-1774: --- Github user jorgebay commented on the issue:

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578497#comment-16578497 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user FlorianHockmann commented on the

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578514#comment-16578514 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/842 I don't know if this matters but I noticed that the upgrade of that JSON thing didn't seem to matter to the tests. they passed on the old version (i sorta tested that version by accident

[GitHub] tinkerpop issue #903: Gremlin.Net: Add ConnectionPool min and max sizes TINK...

Github user jorgebay commented on the issue: https://github.com/apache/tinkerpop/pull/903 I'm back from OOO, I'll look at it this week. ---

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user FlorianHockmann commented on the issue: https://github.com/apache/tinkerpop/pull/842 I'm looking into this. At a first glance it seems to be based on the combination of the upgraded Newtonsoft.Json version and the addition of the .NET Standard 2.0 target. ---

[GitHub] tinkerpop issue #842: TINKERPOP-1945

Github user FlorianHockmann commented on the issue: https://github.com/apache/tinkerpop/pull/842 I think I found the cause of the problem: We're using `dynamic` in Gremlin.Net which requires the `Microsoft.CSharp` package. We always got that package because Newtonsoft.Json also had a

[jira] [Commented] (TINKERPOP-1945) Add support for extended GraphSon types to Gremlin.net

[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578534#comment-16578534 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user FlorianHockmann commented on the

[GitHub] tinkerpop pull request #889: Tinkerpop 1977 - Sasl Authentication

Github user jorgebay commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/889#discussion_r209616766 --- Diff: gremlin-javascript/src/main/javascript/gremlin-javascript/test/integration/sasl-authentication-tests.js --- @@ -0,0 +1,68 @@ +/* + *

[GitHub] tinkerpop pull request #889: Tinkerpop 1977 - Sasl Authentication

Github user jorgebay commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/889#discussion_r209620249 --- Diff: gremlin-javascript/src/main/javascript/gremlin-javascript/lib/driver/auth/sasl-authenticator.js --- @@ -0,0 +1,48 @@ +/* + *

[GitHub] tinkerpop pull request #889: Tinkerpop 1977 - Sasl Authentication

Github user jorgebay commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/889#discussion_r209628120 --- Diff: gremlin-javascript/src/main/javascript/gremlin-javascript/test/integration/sasl-authentication-tests.js --- @@ -0,0 +1,68 @@ +/* + *

[jira] [Commented] (TINKERPOP-2024) Gremlin Server Application archetype should connect via withRemote

[ https://issues.apache.org/jira/browse/TINKERPOP-2024?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578341#comment-16578341 ] ASF GitHub Bot commented on TINKERPOP-2024: --- GitHub user spmallette opened a pull request:

[jira] [Assigned] (TINKERPOP-2024) Gremlin Server Application archetype should connect via withRemote

[ https://issues.apache.org/jira/browse/TINKERPOP-2024?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette reassigned TINKERPOP-2024: --- Assignee: stephen mallette > Gremlin Server Application archetype

[GitHub] tinkerpop issue #908: TINKERPOP-2017 Add check for Column in `by(Function)`

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/908 Cherry picked this commit to tp32 - a0a243b256dbfdf66bb5ce24958fc91c7e6eed03 ---

[jira] [Commented] (TINKERPOP-2017) Check for Column in by()

[ https://issues.apache.org/jira/browse/TINKERPOP-2017?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578349#comment-16578349 ] ASF GitHub Bot commented on TINKERPOP-2017: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop pull request #911: TINKERPOP-2024 Make server archetype use remote...

GitHub user spmallette opened a pull request: https://github.com/apache/tinkerpop/pull/911 TINKERPOP-2024 Make server archetype use remote traversal https://issues.apache.org/jira/browse/TINKERPOP-2024 Since we're promoting remote traversals over scripts it would be better

[GitHub] tinkerpop pull request #908: TINKERPOP-2017 Add check for Column in `by(Func...

Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/908 ---

[jira] [Commented] (TINKERPOP-2017) Check for Column in by()

[ https://issues.apache.org/jira/browse/TINKERPOP-2017?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578351#comment-16578351 ] ASF GitHub Bot commented on TINKERPOP-2017: --- Github user asfgit closed the pull request at: