[jira] [Comment Edited] (TINKERPOP-1254) Support dropping traverser path information when it is no longer needed.

2016-06-16 Thread Ted Wilmes (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15334745#comment-15334745 ] Ted Wilmes edited comment on TINKERPOP-1254 at 6/16/16 10:23 PM: -

[jira] [Commented] (TINKERPOP-1278) Implement Gremlin-Python and general purpose language variant test infrastructure

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

[GitHub] tinkerpop issue #340: add RawExpression to gremlin_python

2016-06-16 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/340 @leifurhauks do you know why travis is not happy? the build took some errors it seems. Can you please look into that as you push your next change? --- If your project is set up for it, you

[GitHub] tinkerpop pull request #342: TINKERPOP-1063 TinkerGraph Performance

2016-06-16 Thread dkuppitz
Github user dkuppitz commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/342#discussion_r67415317 --- Diff: tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraph.java --- @@ -358,6 +350,14 @@ public Features

[jira] [Commented] (TINKERPOP-1063) TinkerGraph performance enhancements

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

[GitHub] tinkerpop issue #340: add RawExpression to gremlin_python

2016-06-16 Thread leifurhauks
Github user leifurhauks commented on the issue: https://github.com/apache/tinkerpop/pull/340 @okram , I realized the current implementation of bindings as dicts is breaking 2/3 compatibility. It also makes it bit hairy to extract just the key in `RawExpression`. I would

[GitHub] tinkerpop issue #342: TINKERPOP-1063 TinkerGraph Performance

2016-06-16 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/342 Do we know that its faster? --- 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

[jira] [Commented] (TINKERPOP-1063) TinkerGraph performance enhancements

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

[GitHub] tinkerpop pull request #342: TINKERPOP-1063 TinkerGraph Performance

2016-06-16 Thread spmallette
GitHub user spmallette opened a pull request: https://github.com/apache/tinkerpop/pull/342 TINKERPOP-1063 TinkerGraph Performance https://issues.apache.org/jira/browse/TINKERPOP-1063 Got rid of `Stream` usage in TinkerGraph vertex/edge iteration. Tests all pass with `mvn

[jira] [Commented] (TINKERPOP-1063) TinkerGraph performance enhancements

2016-06-16 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1063?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15334518#comment-15334518 ] stephen mallette commented on TINKERPOP-1063: - Ended up implementing the latter in the

[jira] [Commented] (TINKERPOP-939) Neo4jGraph should support HighAvailability (Neo4jHA).

2016-06-16 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15334498#comment-15334498 ] stephen mallette commented on TINKERPOP-939: Just for future reference, in case there's a

[GitHub] tinkerpop issue #340: add RawExpression to gremlin_python

2016-06-16 Thread leifurhauks
Github user leifurhauks commented on the issue: https://github.com/apache/tinkerpop/pull/340 > This is just for the GeoPoint-style adding classes? More generally, it can be used to create expressions that make use of APIs, presumably provided by the graph vendor, that aren't

[jira] [Commented] (TINKERPOP-1332) Improve .explain() Dialogue

2016-06-16 Thread Marko A. Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15334420#comment-15334420 ] Marko A. Rodriguez commented on TINKERPOP-1332: --- I now have {{prettyPrint(int

[jira] [Commented] (TINKERPOP-1278) Implement Gremlin-Python and general purpose language variant test infrastructure

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

[GitHub] tinkerpop pull request #340: add RawExpression to gremlin_python

2016-06-16 Thread leifurhauks
GitHub user leifurhauks opened a pull request: https://github.com/apache/tinkerpop/pull/340 add RawExpression to gremlin_python As described on dev@tinkerpop.apache.org thread gremlin_python GLV You can merge this pull request into a Git repository by running: $ git pull

Re: [DISCUSS] Gremlin Language variants in the Gremlin console

2016-06-16 Thread Marko Rodriguez
Yea — submodules is a good idea. Also, I think we should create gremlin-jython, gremlin-ruby, etc. ScriptEngines that have all the imports loaded and ready to go much like we have with gremlin-groovy. Marko. http://markorodriguez.com > On Jun 16, 2016, at 10:19 AM, Stephen Mallette

[jira] [Commented] (TINKERPOP-939) Neo4jGraph should support HighAvailability (Neo4jHA).

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

[GitHub] tinkerpop issue #339: TINKERPOP-939 Added docs for Neo4j HA configuration.

2016-06-16 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/339 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, or if

Re: Gremlin-Python package structure

2016-06-16 Thread Stephen Mallette
> Please make the changes as you see fit and provide a PR. +1 On Thu, Jun 16, 2016 at 12:23 PM, Marko Rodriguez wrote: > HI David, > > Please make the changes as you see fit and provide a PR. I read through > your explanation and it “makes sense” with the limited

Re: Gremlin-Python package structure

2016-06-16 Thread Marko Rodriguez
HI David, Please make the changes as you see fit and provide a PR. I read through your explanation and it “makes sense” with the limited knowledge I have. What I would really like to understand is this whole __init__.py business and how you structure and document/etc. everything. Thank you

[GitHub] tinkerpop issue #336: TINKERPOP-1320 fix GremlinGroovyScriptEngineFileSandbo...

2016-06-16 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/336 `gremlin-core` just have tests that need the some of the same exact functions that are in `gremlin-test`, but the dependencies are such that `gremlin-core` can't use the `TestHelper` in

[DISCUSS] Gremlin Language variants in the Gremlin console

2016-06-16 Thread Daniel Kuppitz
The last few days I've been working on a Gremlin Python code executor for the TinkerPop docs. What I noticed was that we can't simply execute Gremlin Python code in our Gremlin Groovy console, simply because the console has no dependency to the gremlin-variants projects. That was unfortunate for

[GitHub] tinkerpop issue #339: TINKERPOP-939 Added docs for Neo4j HA configuration.

2016-06-16 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/339 Beautiful. Thank you for doing testing that and finally getting that ticket done. Crazy it worked the whole time we just didn't know... VOTE +1. --- If your project is set up for it, you can reply

[jira] [Commented] (TINKERPOP-939) Neo4jGraph should support HighAvailability (Neo4jHA).

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

[jira] [Assigned] (TINKERPOP-939) Neo4jGraph should support HighAvailability (Neo4jHA).

2016-06-16 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-939?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette reassigned TINKERPOP-939: -- Assignee: stephen mallette (was: Daniel Kuppitz) > Neo4jGraph should

[jira] [Closed] (TINKERPOP-1328) Provide [gremlin-python] as an code executor in docs

2016-06-16 Thread Daniel Kuppitz (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1328?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Kuppitz closed TINKERPOP-1328. - Resolution: Fixed Done in branch {{TINKERPOP-1278}} in commit

[jira] [Commented] (TINKERPOP-1320) GremlinGroovyScriptEngineFileSandboxTest throws error: URI is not hierarchical

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

[GitHub] tinkerpop issue #336: TINKERPOP-1320 fix GremlinGroovyScriptEngineFileSandbo...

2016-06-16 Thread pluradj
Github user pluradj commented on the issue: https://github.com/apache/tinkerpop/pull/336 Can you expand on that more, re: the duplication on `TestHelper.java`? I'd bumped into the two classes previously but haven't spent any time trying to figure out the reason why there are two of

[GitHub] tinkerpop issue #336: TINKERPOP-1320 fix GremlinGroovyScriptEngineFileSandbo...

2016-06-16 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/336 I didn't re-test this but both travis builds are good and the code looks right. Just a side note - @pluradj I didn't review this but note that we have a bit of an ugly reality where

[jira] [Commented] (TINKERPOP-1320) GremlinGroovyScriptEngineFileSandboxTest throws error: URI is not hierarchical

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

Re: [DISCUSS] S-Type

2016-06-16 Thread Marko Rodriguez
BTW: Here is GroovyTranslator for Gremlin-Python. https://gist.github.com/okram/406157b21371a540d2641f52a4208760 Realize that Translator exists as a class in both Java and in Python with identical APIs. … and will exist

[jira] [Closed] (TINKERPOP-1301) Provide Javadoc for ScriptInput/OutputFormat's

2016-06-16 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1301?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette closed TINKERPOP-1301. --- Resolution: Done Assignee: stephen mallette Fix Version/s: 3.2.1

[jira] [Commented] (TINKERPOP-1139) [Neo4JGraph] GraphTraversal with SubgraphStrategy removes addLabelStep (as("b"))

2016-06-16 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1139?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15333632#comment-15333632 ] stephen mallette commented on TINKERPOP-1139: - [~mm33] thanks for reporting this - it will