You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tinkerpop.apache.org by "Marko A. Rodriguez (JIRA)" <ji...@apache.org> on 2015/08/01 19:45:04 UTC

[jira] [Commented] (TINKERPOP3-782) map(Traversal) should declare requirements of child

    [ https://issues.apache.org/jira/browse/TINKERPOP3-782?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14650473#comment-14650473 ] 

Marko A. Rodriguez commented on TINKERPOP3-782:
-----------------------------------------------

I fixed this up a bit. Two things:

1. None of the {{TraversalXXXSteps}} have a requirement save for what their children require. Thus:

{code}
    @Override
    public Set<TraverserRequirement> getRequirements() {
        return this.getSelfAndChildRequirements();
    }
{code}

2. You only did {{TraversalMapStep}} and {{TraversalFlatMapStep}}. I added the above to {{TraversalSideEffectStep}} and {{TraversalFilterStep}}.

> map(Traversal) should declare requirements of child
> ---------------------------------------------------
>
>                 Key: TINKERPOP3-782
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP3-782
>             Project: TinkerPop 3
>          Issue Type: Bug
>          Components: process
>    Affects Versions: 3.0.0-incubating
>            Reporter: Matt Frantz
>            Assignee: Matt Frantz
>             Fix For: 3.0.1-incubating, 3.1.0-incubating
>
>
> {noformat}
> gremlin> g.V().as('a').map(select('a'))
> gremlin> g.withPath().V().as('a').map(select('a'))
> ==>v[1]
> ==>v[2]
> ==>v[3]
> ==>v[4]
> ==>v[5]
> ==>v[6]
> {noformat}
> Also applies to {{flatMap(Traversal)}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)