[jira] [Commented] (TINKERPOP-1541) Select should default to Pop.last semantics

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

[GitHub] tinkerpop issue #482: TINKERPOP-1541: Select should default to Pop.last sema...

Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/482 `docker/build.sh -t -i -n` succeeded. 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

[GitHub] tinkerpop issue #486: TINKERPOP-1545 IncidentToAdjacentStrategy is buggy

Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/486 Force-pushed. Filtering steps based on 2 conditions (type and traversal requirements) was easier than I thought (thanks to `TraversalHelper.anyStepRecursively()`). --- If your project is set

[jira] [Commented] (TINKERPOP-1545) IncidentToAdjacentStrategy is buggy

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

[jira] [Commented] (TINKERPOP-1545) IncidentToAdjacentStrategy is buggy

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

[GitHub] tinkerpop issue #486: TINKERPOP-1545 IncidentToAdjacentStrategy is buggy

Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/486 You know what, I will revert the change and not implement `PathProcessor` in `SimplePathStep`. Here's why: `IncidentToAdjacentStrategy` should only rely on the `getRequirements()` result,

[jira] [Commented] (TINKERPOP-1545) IncidentToAdjacentStrategy is buggy

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

[GitHub] tinkerpop issue #486: TINKERPOP-1545 IncidentToAdjacentStrategy is buggy

Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/486 The `PathProcessor` interface has no function for SimpleStep and doesn't add any overhead. AFAIK `PathProcessor` is only used by strategies to easily identify traversals with "special

[GitHub] tinkerpop pull request #478: TINKERPOP-1490 Implemented promise API for Trav...

Github user al3xandru commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/478#discussion_r87521066 --- Diff: gremlin-core/pom.xml --- @@ -61,6 +61,11 @@ limitations under the License. +

[GitHub] tinkerpop issue #486: TINKERPOP-1545 IncidentToAdjacentStrategy is buggy

Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/486 `PathProcessors` are intended for path analysis steps that use `by()`-modulation. Also, what about `CyclicPathStep`? Perhaps just itemize those steps explictly for now and now have them implement

[jira] [Commented] (TINKERPOP-1547) Two bugs found associated with MatchStep: Path retraction and range count.

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

[GitHub] tinkerpop pull request #483: TINKERPOP-1547: Two bugs found associated with ...

Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/483 --- 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] [Closed] (TINKERPOP-1547) Two bugs found associated with MatchStep: Path retraction and range count.

[ https://issues.apache.org/jira/browse/TINKERPOP-1547?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marko A. Rodriguez closed TINKERPOP-1547. - Resolution: Fixed > Two bugs found associated with MatchStep: Path retraction

[GitHub] tinkerpop pull request #486: TINKERPOP-1545 IncidentToAdjacentStrategy is bu...

GitHub user dkuppitz opened a pull request: https://github.com/apache/tinkerpop/pull/486 TINKERPOP-1545 IncidentToAdjacentStrategy is buggy https://issues.apache.org/jira/browse/TINKERPOP-1545 Made `SimplePathStep` implement `PathProcessor` and `IncidentToAdjacentStrategy`

[jira] [Commented] (TINKERPOP-1545) IncidentToAdjacentStrategy is buggy

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

[jira] [Assigned] (TINKERPOP-1545) IncidentToAdjacentStrategy is buggy

[ https://issues.apache.org/jira/browse/TINKERPOP-1545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Kuppitz reassigned TINKERPOP-1545: - Assignee: Daniel Kuppitz > IncidentToAdjacentStrategy is buggy >

[jira] [Commented] (TINKERPOP-1490) Provider a Future based Traversal.async(Function<Traversal,V>) terminal step

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

[GitHub] tinkerpop issue #478: TINKERPOP-1490 Implemented promise API for Traversal

Github user aholmberg commented on the issue: https://github.com/apache/tinkerpop/pull/478 > the type of future returned will depend on the underlying RemoteConnection implementation. That depends on whether you are okay leaking remote connection implementation through this

Re: [DISCUSS] Add HGraphDB to graph system listing

I sorta share that sentiment however, I think we had that discussion back when we determined our listing requirements and we figured we couldn't figure out how to easily quantify "mature". On Thu, Nov 10, 2016 at 3:17 PM, Jean-Baptiste Musso wrote: > Interesting thing. +1 to

Re: [DISCUSS] Add HGraphDB to graph system listing

Interesting thing. +1 to adding HGraphDB to the list. Minor concern: I'm not sure if we want to manage a curated list of implementations and maybe restrict to more mature projects. I'm just thinking out loud here: I'm not targeting HGraphDB specifically (and I wish them good luck - *cheers*). My

Add HGraphDB to the homepage index?

Hi, I wrote a TinkerPop 3 provider for HBase so it could be used as an OLTP graph database. Would it be possible to get it added to the homepage index listing? https://github.com/rayokota/hgraphdb https://rayokota.wordpress.com/2016/11/10/hgraphdb-hbase- as-a-tinkerpop-graph-database/ Thanks,

[jira] [Commented] (TINKERPOP-1490) Provider a Future based Traversal.async(Function<Traversal,V>) terminal step

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

[GitHub] tinkerpop issue #478: TINKERPOP-1490 Implemented promise API for Traversal

Github user aholmberg commented on the issue: https://github.com/apache/tinkerpop/pull/478 Coupling the traversal API to a web framework does not seem appealing to me. Could we consider using the [futures backport](https://pypi.python.org/pypi/futures), which backports the Python 3

[jira] [Commented] (TINKERPOP-1541) Select should default to Pop.last semantics

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

[GitHub] tinkerpop issue #482: TINKERPOP-1541: Select should default to Pop.last sema...

Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/482 I'm cool with that change. I still have to run a `docker/build.sh -t -i -n` though - will do so as soon as I have the resources available (unless someone else volunteers). --- If your project

[jira] [Commented] (TINKERPOP-1541) Select should default to Pop.last semantics

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

[GitHub] tinkerpop issue #482: TINKERPOP-1541: Select should default to Pop.last sema...

Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/482 When I go to merge, I will change the method name to `selectV3d2()` so if you want to vote :) ... Also, I think we need @dkuppitz and @robertdale and @mhfrantz to have a look and vote too.

[jira] [Created] (TINKERPOP-1549) Add more RangeStep test cases

Daniel Kuppitz created TINKERPOP-1549: - Summary: Add more RangeStep test cases Key: TINKERPOP-1549 URL: https://issues.apache.org/jira/browse/TINKERPOP-1549 Project: TinkerPop Issue

[jira] [Created] (TINKERPOP-1548) Traversals can complete before interrupted in TraversalInterruptionComputerTest

Ted Wilmes created TINKERPOP-1548: - Summary: Traversals can complete before interrupted in TraversalInterruptionComputerTest Key: TINKERPOP-1548 URL: https://issues.apache.org/jira/browse/TINKERPOP-1548

[jira] [Commented] (TINKERPOP-1547) Two bugs found associated with MatchStep: Path retraction and range count.

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

[GitHub] tinkerpop issue #483: TINKERPOP-1547: Two bugs found associated with MatchSt...

Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/483 @dkuppitz -- note that @spmallette never VOTE'd so when you are done with your testing and updates, just VOTE and I will merge. --- If your project is set up for it, you can reply to this email

[jira] [Assigned] (TINKERPOP-1428) profile() throws NPE for union(group, group)

[ https://issues.apache.org/jira/browse/TINKERPOP-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marko A. Rodriguez reassigned TINKERPOP-1428: - Assignee: Marko A. Rodriguez > profile() throws NPE for union(group,

[jira] [Closed] (TINKERPOP-1428) profile() throws NPE for union(group, group)

[ https://issues.apache.org/jira/browse/TINKERPOP-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Marko A. Rodriguez closed TINKERPOP-1428. - Resolution: Fixed Fix Version/s: 3.2.4 > profile() throws NPE for

[jira] [Commented] (TINKERPOP-1428) profile() throws NPE for union(group, group)

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

[jira] [Commented] (TINKERPOP-1490) Provider a Future based Traversal.async(Function<Traversal,V>) terminal step

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

[GitHub] tinkerpop issue #478: TINKERPOP-1490 Implemented promise API for Traversal

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/478 thanks @davebshow - i might ask for your assistance on implementing that. still trying to get it right for java. that blocking call in `RemoteStep` stinks. having a bit of a hard

[GitHub] tinkerpop pull request #481: TINKERPOP-1428: profile() throws NPE for union(...

Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/481 --- 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-1547) Two bugs found associated with MatchStep: Path retraction and range count.

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

[GitHub] tinkerpop issue #483: TINKERPOP-1547: Two bugs found associated with MatchSt...

Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/483 Please don't merge this PR yet. While testing the changes in `RangeByIsCountStrategy` I realized that we can add a minor tweak to get another great optimization. I'll push my local changes as

[jira] [Commented] (TINKERPOP-1490) Provider a Future based Traversal.async(Function<Traversal,V>) terminal step

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

[GitHub] tinkerpop issue #478: TINKERPOP-1490 Implemented promise API for Traversal

Github user davebshow commented on the issue: https://github.com/apache/tinkerpop/pull/478 @spmallette you can easily implement something similar in Python using Futures. Using `tornado.concurrent.Future` would probably make the most sense, as is is already 2/3 compatible and should

[jira] [Commented] (TINKERPOP-1541) Select should default to Pop.last semantics

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

[GitHub] tinkerpop issue #482: TINKERPOP-1541: Select should default to Pop.last sema...

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/482 ok - if it's trivial, then perhaps not worrying about. --- 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

[jira] [Commented] (TINKERPOP-1541) Select should default to Pop.last semantics

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

[GitHub] tinkerpop issue #482: TINKERPOP-1541: Select should default to Pop.last sema...

Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/482 The only difference is `select(Pop.mix,...)` (that is `selectV3d2()`) and there are 6 or so tests that test it in `RangeTest` and `TailTest`. I can explicitly make them use the method

[jira] [Commented] (TINKERPOP-1547) Two bugs found associated with MatchStep: Path retraction and range count.

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

[GitHub] tinkerpop issue #483: TINKERPOP-1547: Two bugs found associated with MatchSt...

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/483 @twilmes can you please send some specifics on what interruption tests were tanking on you and under what circumstances? Maybe just ping me on hipchat when you have a moment rather than go

[jira] [Commented] (TINKERPOP-1541) Select should default to Pop.last semantics

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

[GitHub] tinkerpop issue #482: TINKERPOP-1541: Select should default to Pop.last sema...

Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/482 It should be `selectV3d2()` given that it will yield backward compatibility to 3.2.x. Maybe I missed it, but are there any tests for `selectV3d2()` - at least something to exercise

[jira] [Commented] (TINKERPOP-1428) profile() throws NPE for union(group, group)

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