[GitHub] tinkerpop issue #323: TINKERPOP-1298 Save OLAP results to file

2016-06-15 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/323 Closing the PR, but I'll keep the branch alive. --- 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

[jira] [Commented] (TINKERPOP-1298) Save OLAP results to file

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

[GitHub] tinkerpop pull request #323: TINKERPOP-1298 Save OLAP results to file

2016-06-15 Thread dkuppitz
Github user dkuppitz closed the pull request at: https://github.com/apache/tinkerpop/pull/323 --- 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

[GitHub] tinkerpop issue #337: TINKERPOP-1332: Improve .explain() Dialogue

2016-06-15 Thread RussellSpitzer
Github user RussellSpitzer commented on the issue: https://github.com/apache/tinkerpop/pull/337 I still think this is pretty verbose, but I'm happy with the inclusion of the ```graphfilter[[VertexStep(OUT,edge)]]``` so +1 on that --- If your project is set up for it, you can

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

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

[GitHub] tinkerpop pull request #333: if there is no edge label in the GraphML file, ...

2016-06-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/333 --- 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

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

2016-06-15 Thread Marko A. Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15331610#comment-15331610 ] Marko A. Rodriguez commented on TINKERPOP-1332: --- I have done two things. 1.

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

2016-06-15 Thread Marko A. Rodriguez (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marko A. Rodriguez reassigned TINKERPOP-1332: - Assignee: Marko A. Rodriguez > Improve .explain() Dialogue >

[jira] [Commented] (TINKERPOP-1319) several FeatureRequirement annotations are incorrect in gremlin-test

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

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

2016-06-15 Thread Bob Briody (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15331802#comment-15331802 ] Bob Briody commented on TINKERPOP-1332: --- I don't like the idea of introducing word wrap here.

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

2016-06-15 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15331811#comment-15331811 ] stephen mallette commented on TINKERPOP-1332: - I sort of agree with [~rjbriody] about

[DISCUSS] S-Type

2016-06-15 Thread Marko Rodriguez
Hello, I think we should introduce an S object. It would stand for Script and would allow you to do: S.f(“{ it.length }”) Where does this come in handy? Language variants. Imagine using gremlin_python that will compile to Gremlin-Groovy for execution at GremlinServer.

[GitHub] tinkerpop pull request #336: TINKERPOP-1320 fix GremlinGroovyScriptEngineFil...

2016-06-15 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/336#discussion_r67157460 --- Diff: gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoWriter.java --- @@ -131,7 +131,7 @@ public void

[jira] [Commented] (TINKERPOP-1319) several FeatureRequirement annotations are incorrect in gremlin-test

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

[GitHub] tinkerpop pull request #335: TINKERPOP-1319: fix incorrect FeatureRequiremen...

2016-06-15 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/335#discussion_r67159933 --- Diff: gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/FeatureSupportTest.java --- @@ -587,7 +587,7 @@ public void

[GitHub] tinkerpop pull request #337: TINKERPOP-1332: Improve .explain() Dialogue

2016-06-15 Thread okram
GitHub user okram opened a pull request: https://github.com/apache/tinkerpop/pull/337 TINKERPOP-1332: Improve .explain() Dialogue https://issues.apache.org/jira/browse/TINKERPOP-1332 `TraversalExplanation.toString()` now supports word wrapping and `GraphFilter`

[jira] [Commented] (TINKERPOP-1319) several FeatureRequirement annotations are incorrect in gremlin-test

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

[GitHub] tinkerpop issue #315: Remove dashes from labels and property keys

2016-06-15 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/315 @analytically did you happen to see my previous comment about rebasing/fixing conflicts on this PR so that we can begin code review/testing on our end? is that something you have some time to

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

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

[GitHub] tinkerpop issue #335: TINKERPOP-1319: fix incorrect FeatureRequirement annot...

2016-06-15 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/335 VOTE +1 - pending update to upgrade docs --- 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

[GitHub] tinkerpop pull request #336: TINKERPOP-1320 fix GremlinGroovyScriptEngineFil...

2016-06-15 Thread spmallette
Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/336#discussion_r67174102 --- Diff: gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoWriter.java --- @@ -131,7 +131,7 @@ public void

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

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

[GitHub] tinkerpop issue #338: TINKERPOP-1139 Re-assigned step labels as appropriate ...

2016-06-15 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/338 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

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

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

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

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

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

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

[jira] [Closed] (TINKERPOP-1196) Calls to Result.one() might block indefinitely

2016-06-15 Thread stephen mallette (JIRA)
[ https://issues.apache.org/jira/browse/TINKERPOP-1196?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stephen mallette closed TINKERPOP-1196. --- Resolution: Fixed Fix Version/s: 3.2.1 This PR was merged a while ago -

Re: gremlin_python GLV

2016-06-15 Thread Marko Rodriguez
Hello, > I was reading through the gremlin_python GLV code this morning, and > overall it looks like it should work pretty smoothly. Thanks for doing > this Marko, really cool work! I just wanted to comment on a few things > that popped out at me on my first reading.The major issue I see is >

[GitHub] tinkerpop issue #338: TINKERPOP-1139 Re-assigned step labels as appropriate ...

2016-06-15 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/338 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 the