[jira] [Commented] (TINKERPOP-2019) Gremlin.Net.Driver.WebSocketConnection throws System.InvalidOperationException

2018-08-15 Thread Florian Hockmann (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-2019?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16580916#comment-16580916 ] Florian Hockmann commented on TINKERPOP-2019: - Ok, now we're talking about two problems

Re: [DISCUSS] type safer GLVs

2018-08-15 Thread Stephen Mallette
Hi Yura, welcome to the dev list and thanks for offering this thought for discussion. > wouldn't it be nice to make Gremlin (or rather GLVs) a bit more type safe? You mention GLVs and I'd say that the choice to be more type safe would rest on the level of type safety features the language

Re: [VOTE] Harsh Thakkar for Committer

2018-08-15 Thread Stephen Mallette
dah.please ignore..that wasn't meant for dev list. i've spilled the beans... On Wed, Aug 15, 2018 at 7:51 AM Stephen Mallette wrote: > Given that there were no objections to Harsh as a committer in our > discussions earlier, let's do the VOTEing thing - I'm a +1. > > This vote will close

[DISCUSS] Add Gremlinq to Index Listing

2018-08-15 Thread Stephen Mallette
We recently learned of Gremlinq https://github.com/ExRam/ExRam.Gremlinq while dealing with a nice pull request from Daniel Weber. I believe that it now meets our requirements for the index listing. If there are no objections in the next 72 hours, I will assume lazy consensus and move forward to

[DISCUSS] Add StackState to Powered By Listing

2018-08-15 Thread Stephen Mallette
StackState uses TinkerPop: https://docs.stackstate.com/guides/queries/ Gremlin is exposed directly as a query language for analysis of the data that it collects. I think we could add them to the "Powered By" listing. I requested that they add Apache TinkerPop™ to

[jira] [Updated] (TINKERPOP-2026) Gremlin.Net.Driver should check ClientWebSocket.State before closing

2018-08-15 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-2026?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette updated TINKERPOP-2026: Affects Version/s: 3.2.9 Component/s: dotnet > Gremlin.Net.Driver

Re: [DISCUSS] Options for withRemote()

2018-08-15 Thread Robert Dale
Are you thinking of another mid-stream change or something for TinkerPop 3.4 or 4.0? Robert Dale On Mon, Aug 13, 2018 at 6:35 PM Stephen Mallette wrote: > Yes - I did consider killing instantiation through a Graph except that: > > 1. It felt like a major major departure from what we have been

[GitHub] tinkerpop issue #842: TINKERPOP-1945

2018-08-15 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/842 I proposed the listing on the dev list...the preview release is nice, but could you reference it in your README so that it's clear. Also, please include mention of "Apache TinkerPop™" in

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

2018-08-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16580963#comment-16580963 ] 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

2018-08-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16581084#comment-16581084 ] ASF GitHub Bot commented on TINKERPOP-1945: --- Github user danielcweber commented on the

[GitHub] tinkerpop issue #842: TINKERPOP-1945

2018-08-15 Thread danielcweber
Github user danielcweber commented on the issue: https://github.com/apache/tinkerpop/pull/842 Done! ---

[GitHub] tinkerpop issue #914: Do not format and reparse eval timeout arg when evalua...

2018-08-15 Thread newkek
Github user newkek commented on the issue: https://github.com/apache/tinkerpop/pull/914 I've pushed a fix for tests that used to send the Eval_timeout as an int instead of a long, but also made the server robust in case an int is still sent. Not sure if we want to do that or to

Re: [DISCUSS] Options for withRemote()

2018-08-15 Thread Robert Dale
I liked RemoteGraph. It was fairly concise in what it was. It also kept things in line with other graphs, i.e. XGraph.open() (perhaps due to the fact that they all had their roots here). EmptyGraph is kinda weird and verbose [EmptyGraph.instance().traversal().withRemote()] I don't like

[VOTE] Harsh Thakkar for Committer

2018-08-15 Thread Stephen Mallette
Given that there were no objections to Harsh as a committer in our discussions earlier, let's do the VOTEing thing - I'm a +1. This vote will close August 18th 2018 at 8am EST

[jira] [Commented] (TINKERPOP-1906) Make ResponseException explorable

2018-08-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1906?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16581684#comment-16581684 ] ASF GitHub Bot commented on TINKERPOP-1906: --- Github user redoz commented on the issue:

[GitHub] tinkerpop issue #915: Tinkerpop 1913-Followup

2018-08-15 Thread redoz
Github user redoz commented on the issue: https://github.com/apache/tinkerpop/pull/915 Is it out of scope to add the attribute collection to the ResponseException as well? It's a very minor change (it requires only adding the property to the exception type, and changing one line of

[GitHub] tinkerpop pull request #915: Tinkerpop 1913-Followup

2018-08-15 Thread redoz
Github user redoz commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/915#discussion_r210420704 --- Diff: gremlin-dotnet/src/Gremlin.Net/Driver/IResultSet.cs --- @@ -0,0 +1,43 @@ +#region License +/* + * Licensed to the Apache Software