[jira] [Commented] (TINKERPOP-1539) Create a ComplexTraversalTest with crazy nested gnarly traversals.

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15714043#comment-15714043 ] ASF GitHub Bot commented on TINKERPOP-1539: --- GitHub user dkuppitz opened a pull request:

[GitHub] tinkerpop pull request #511: TINKERPOP-1539 Create a ComplexTraversalTest wi...

2016-12-01 Thread dkuppitz
GitHub user dkuppitz opened a pull request: https://github.com/apache/tinkerpop/pull/511 TINKERPOP-1539 Create a ComplexTraversalTest with crazy nested gnarly traversals. https://issues.apache.org/jira/browse/TINKERPOP-1539 Added 2 more tests, one for OLTP only and one for

[jira] [Commented] (TINKERPOP-1493) Groovy project doesn't build on Windows

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15713832#comment-15713832 ] ASF GitHub Bot commented on TINKERPOP-1493: --- Github user asfgit closed the pull request at:

[GitHub] tinkerpop pull request #457: TINKERPOP-1493 Groovy project doesn't build on ...

2016-12-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/457 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] tinkerpop pull request #508: TINKERPOP-1363 Cleanup Docker build script for ...

2016-12-01 Thread dkuppitz
Github user dkuppitz commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/508#discussion_r90561817 --- Diff: CHANGELOG.asciidoc --- @@ -26,6 +26,7 @@ image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima TinkerPop 3.3.0

[jira] [Commented] (TINKERPOP-1363) Cleanup Docker build script for next major release

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15713433#comment-15713433 ] ASF GitHub Bot commented on TINKERPOP-1363: --- Github user dkuppitz commented on a diff in the

Re: [TinkerPop] gremlin-x

2016-12-01 Thread Stephen Mallette
I don't think HaltedTraverserStrategy is a good way to deal with properties vs no properties. It really should be a function of the serializer. I say that because I seem to remember that if the Vertex (or Edge or whatever) is embedded in another object then the HaltedTraverserStrategy won't

Updating Gremlin's Wikipedia page.

2016-12-01 Thread Marko Rodriguez
Hi, Anyone interested in working on Gremlin’s Wikipedia page? https://en.wikipedia.org/wiki/Gremlin_(programming_language) Discussing Gremlin bytecode would be good. Bye, Marko. http://markorodriguez.com

[jira] [Commented] (TINKERPOP-1565) GraphSON 2.0 updates -- supporting attachment, maintaining consistency, and reducing verbosity

2016-12-01 Thread Marko A. Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1565?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15713071#comment-15713071 ] Marko A. Rodriguez commented on TINKERPOP-1565: --- Here is an idea that I had with

[jira] [Commented] (TINKERPOP-1576) gremlin-python calls non-existent methods

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15712969#comment-15712969 ] ASF GitHub Bot commented on TINKERPOP-1576: --- Github user FlorianHockmann commented on the

[GitHub] tinkerpop issue #509: TINKERPOP-1576 gremlin-python calls non-existent metho...

2016-12-01 Thread FlorianHockmann
Github user FlorianHockmann commented on the issue: https://github.com/apache/tinkerpop/pull/509 Yeah, `start()` could really lead to problems in client applications. I will remove it from the statics. --- If your project is set up for it, you can reply to this email and have your

[jira] [Commented] (TINKERPOP-1576) gremlin-python calls non-existent methods

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15712959#comment-15712959 ] ASF GitHub Bot commented on TINKERPOP-1576: --- Github user okram commented on the issue:

[GitHub] tinkerpop issue #509: TINKERPOP-1576 gremlin-python calls non-existent metho...

2016-12-01 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/509 I would actually remove `start()` from static imports. Its never used as its more of a "helper method" and then less chance for naming collisions. --- If your project is set up for it, you can

[jira] [Commented] (TINKERPOP-1576) gremlin-python calls non-existent methods

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15712949#comment-15712949 ] ASF GitHub Bot commented on TINKERPOP-1576: --- Github user okram commented on the issue:

[jira] [Commented] (TINKERPOP-1576) gremlin-python calls non-existent methods

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15712951#comment-15712951 ] ASF GitHub Bot commented on TINKERPOP-1576: --- Github user FlorianHockmann commented on the

[GitHub] tinkerpop issue #509: TINKERPOP-1576 gremlin-python calls non-existent metho...

2016-12-01 Thread FlorianHockmann
Github user FlorianHockmann commented on the issue: https://github.com/apache/tinkerpop/pull/509 Ok, I wasn't sure whether those methods make sense for a GLV, but it's good to know as I can now also implement it that way for the C# GLV. My last commit adds hard-coded

[GitHub] tinkerpop issue #509: TINKERPOP-1576 gremlin-python calls non-existent metho...

2016-12-01 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/509 Perfect. VOTE +1. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

Re: [TinkerPop] gremlin-x

2016-12-01 Thread pieter-gmail
Hi, "So with ReferenceElements, latency will be less too because it takes less time to construct the ReferenceVertex than it does to construct a DetachedVertex. Imagine a vertex with 100 properties and meta properties. ?!" But ReferencedElement does not have the properties so more round trips

[jira] [Updated] (TINKERPOP-1565) GraphSON 2.0 updates -- supporting attachment, maintaining consistency, and reducing verbosity

2016-12-01 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1565?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette updated TINKERPOP-1565: Fix Version/s: (was: 3.2.3) The fix version was set to 3.2.3 - removed

[jira] [Commented] (TINKERPOP-1493) Groovy project doesn't build on Windows

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15712732#comment-15712732 ] ASF GitHub Bot commented on TINKERPOP-1493: --- Github user pluradj commented on the issue:

[GitHub] tinkerpop issue #457: TINKERPOP-1493 Groovy project doesn't build on Windows

2016-12-01 Thread pluradj
Github user pluradj commented on the issue: https://github.com/apache/tinkerpop/pull/457 We have enough votes to commit this. I'll handle it today. Thanks everyone. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[jira] [Commented] (TINKERPOP-1363) Cleanup Docker build script for next major release

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15712720#comment-15712720 ] ASF GitHub Bot commented on TINKERPOP-1363: --- Github user pluradj commented on the issue:

[GitHub] tinkerpop issue #508: TINKERPOP-1363 Cleanup Docker build script for next ma...

2016-12-01 Thread pluradj
Github user pluradj commented on the issue: https://github.com/apache/tinkerpop/pull/508 `docker/build.sh -t -i -n -d` worked for me. I ran a Docker build against `tp31` afterwards, and that worked too. VOTE: +1 --- If your project is set up for it, you can reply to this

[jira] [Commented] (TINKERPOP-1363) Cleanup Docker build script for next major release

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15712708#comment-15712708 ] ASF GitHub Bot commented on TINKERPOP-1363: --- Github user pluradj commented on a diff in the

[GitHub] tinkerpop pull request #508: TINKERPOP-1363 Cleanup Docker build script for ...

2016-12-01 Thread pluradj
Github user pluradj commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/508#discussion_r90509684 --- Diff: CHANGELOG.asciidoc --- @@ -26,6 +26,7 @@ image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima TinkerPop 3.3.0

Fwd: [TinkerPop] gremlin-x

2016-12-01 Thread Marko Rodriguez
Sorry, someone took out dev@ in the original email of my reply below. Marko. http://markorodriguez.com > Begin forwarded message: > > From: Marko Rodriguez > Subject: Re: [TinkerPop] gremlin-x > Date: December 1, 2016 at 11:38:21 AM MST > To:

Re: [TinkerPop] gremlin-x

2016-12-01 Thread Stephen Mallette
adding back dev With Pieter's points in mind, Marko, ultimately does the "classic" embedded mode to using TinkerPop go away for JVM users? I mean - it's not as though those classes disappear. I suppose that in the end there's nothing we can really do to prevent someone on the JVM from

Re: [TinkerPop] gremlin-x

2016-12-01 Thread Stephen Mallette
> Something that should help performance and might help get away from general scripts is batching I've thought about batching as a feature many times, but i'm not sure it solves the use case folks typically have that causes us problems. If most batches looked like the one in your example that

[jira] [Closed] (TINKERPOP-1580) Resolve naming collisions for static imports in the Gremlin language

2016-12-01 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1580?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette closed TINKERPOP-1580. --- Resolution: Duplicate > Resolve naming collisions for static imports in the

[jira] [Created] (TINKERPOP-1579) Resolve naming collisions for static imports in the Gremlin language

2016-12-01 Thread stephen mallette (JIRA)
stephen mallette created TINKERPOP-1579: --- Summary: Resolve naming collisions for static imports in the Gremlin language Key: TINKERPOP-1579 URL: https://issues.apache.org/jira/browse/TINKERPOP-1579

[jira] [Created] (TINKERPOP-1580) Resolve naming collisions for static imports in the Gremlin language

2016-12-01 Thread stephen mallette (JIRA)
stephen mallette created TINKERPOP-1580: --- Summary: Resolve naming collisions for static imports in the Gremlin language Key: TINKERPOP-1580 URL: https://issues.apache.org/jira/browse/TINKERPOP-1580

Re: [TinkerPop] gremlin-x

2016-12-01 Thread Robert Dale
As someone who has gone through the trouble to migrate all scripts to Java withRemote, I'll throw in my $0.02. I agree the focus should be on the Connection (being separate from Graph) and Traversal. I wouldn't constrain it to "RemoteConnection", just Connection or GraphConnection. Perhaps

[jira] [Commented] (TINKERPOP-1578) Support writable distributed graphs using spark as the underlying engine

2016-12-01 Thread Dovid Kopel (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15712441#comment-15712441 ] Dovid Kopel commented on TINKERPOP-1578: So far I haven't touched graph computer at

[jira] [Commented] (TINKERPOP-1363) Cleanup Docker build script for next major release

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15712397#comment-15712397 ] ASF GitHub Bot commented on TINKERPOP-1363: --- Github user twilmes commented on the issue:

[GitHub] tinkerpop issue #508: TINKERPOP-1363 Cleanup Docker build script for next ma...

2016-12-01 Thread twilmes
Github user twilmes commented on the issue: https://github.com/apache/tinkerpop/pull/508 `docker/build.sh -t -i -n -d` passed and I took a look at the docs and confirmed they generated correctly. VOTE: +1 --- If your project is set up for it, you can reply to this email and

[jira] [Commented] (TINKERPOP-1562) Migrate ScriptEngine-related code to gremlin-core

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15712325#comment-15712325 ] ASF GitHub Bot commented on TINKERPOP-1562: --- Github user spmallette commented on the issue:

[GitHub] tinkerpop issue #510: TINKERPOP-1562 Migrate ScriptEngine-related code to gr...

2016-12-01 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/510 @okram one thing that could use your explict review here - we lost the ability to do wildcard imports given the work you did on `CoreImports` for gremlin-python. Some plugins used a lot of

Re: [TinkerPop] gremlin-x

2016-12-01 Thread Marko Rodriguez
Hello, > You say "v1.property(‘name’).value()" should never happen. > This is something we use all the time. Almost all the work we do is with > vertices and edges and is natural to call > element.property("something").isPresent(), > element.property("something").value(),

[GitHub] tinkerpop pull request #510: TINKERPOP-1562 Migrate ScriptEngine-related cod...

2016-12-01 Thread spmallette
GitHub user spmallette opened a pull request: https://github.com/apache/tinkerpop/pull/510 TINKERPOP-1562 Migrate ScriptEngine-related code to gremlin-core https://issues.apache.org/jira/browse/TINKERPOP-1562 Apologies in advance - this is a fat, nasty PR and I doubt there

Re: [TinkerPop] gremlin-x

2016-12-01 Thread Jason Plurad
> + I'll come back to this in the points below, but as a big fat general question: how will we deal with transactions? Bullseye. I'd add that assuming that TinkerGraph would be the "one true TinkerPop graph" for gremlin-x testing, it needs to support transactions which it doesn't do currently. Or

Re: [TinkerPop] gremlin-x

2016-12-01 Thread pieter-gmail
Hi, You say "v1.property(‘name’).value()" should never happen. This is something we use all the time. Almost all the work we do is with vertices and edges and is natural to call element.property("something").isPresent(), element.property("something").value(), element.property("something",

Re: [TinkerPop] gremlin-x

2016-12-01 Thread Marko Rodriguez
Hi, > I think we've tried a lot of things for TinkerPop 3 and that there are too > many options for application development as well as an over-exposed API, so I > like this direction, though a fair bit of complexity lies in making it > happen. Before getting to that complexity, I don't really

[jira] [Closed] (TINKERPOP-1573) Bindings don't work in coalesce

2016-12-01 Thread Marko A. Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marko A. Rodriguez closed TINKERPOP-1573. - Resolution: Fixed Fix Version/s: 3.2.4 > Bindings don't work in

[GitHub] tinkerpop issue #507: TINKERPOP-1573: Bindings don't work in coalesce

2016-12-01 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/507 Yea, go ahead and merge it. One day I'll be bored and rewrite this stuff. Maybe around Xmas. --- If your project is set up for it, you can reply to this email and have your reply appear on

[jira] [Commented] (TINKERPOP-1573) Bindings don't work in coalesce

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15711928#comment-15711928 ] ASF GitHub Bot commented on TINKERPOP-1573: --- Github user okram commented on the issue:

[GitHub] tinkerpop issue #507: TINKERPOP-1573: Bindings don't work in coalesce

2016-12-01 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/507 I know it sucks. I don't like this any more than you do. Here is the saving grace -- if someone comes up with a better solution, there are only two classes we need to tweak (`Bytecode` and

[jira] [Commented] (TINKERPOP-1573) Bindings don't work in coalesce

2016-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15711914#comment-15711914 ] ASF GitHub Bot commented on TINKERPOP-1573: --- Github user dkuppitz commented on the issue:

[jira] [Commented] (TINKERPOP-1578) Support writable distributed graphs using spark as the underlying engine

2016-12-01 Thread Marko A. Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15711896#comment-15711896 ] Marko A. Rodriguez commented on TINKERPOP-1578: --- Can you generalize to make work for

[jira] [Commented] (TINKERPOP-1539) Create a ComplexTraversalTest with crazy nested gnarly traversals.

2016-12-01 Thread Marko A. Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15711890#comment-15711890 ] Marko A. Rodriguez commented on TINKERPOP-1539: --- As a side, I do think we should add a

[jira] [Updated] (TINKERPOP-1578) Support writable distributed graphs using spark as the underlying engine

2016-12-01 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1578?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette updated TINKERPOP-1578: Affects Version/s: 3.2.3 Component/s: hadoop Issue Type:

[jira] [Commented] (TINKERPOP-1539) Create a ComplexTraversalTest with crazy nested gnarly traversals.

2016-12-01 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15711859#comment-15711859 ] stephen mallette commented on TINKERPOP-1539: - I sense that we could use some additional

[jira] [Commented] (TINKERPOP-1539) Create a ComplexTraversalTest with crazy nested gnarly traversals.

2016-12-01 Thread Marko A. Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1539?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15711860#comment-15711860 ] Marko A. Rodriguez commented on TINKERPOP-1539: --- The problem with creating new graphs in