[ 
https://issues.apache.org/jira/browse/TINKERPOP-1243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Marko A. Rodriguez closed TINKERPOP-1243.
-----------------------------------------
    Resolution: Invalid
      Assignee: Marko A. Rodriguez

This is a nice idea, but the problem, {{MatchStep}} is actually a 
{{FlatMapStep}} as it "flatmaps out" pattern matches given the input. If we are 
going to do anything in the spirit of this ticket, it should be that the input 
is returned, NOT a {{Map<String,Object>}} (that is what {{select()}} is for). I 
will close this ticket as invalid and if we wish later on, we can remove the 
{{Map<String,Object>}} mapping in another ticket.

> MatchStep should be a FilterStep, not a MapStep
> -----------------------------------------------
>
>                 Key: TINKERPOP-1243
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1243
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.1.1-incubating, 3.2.0-incubating
>            Reporter: Marko A. Rodriguez
>            Assignee: Marko A. Rodriguez
>              Labels: breaking
>
> {{MatchStep}} should really be a {{FilterStep}}. It simply filters out all 
> traversers that don't match the patterns. However, given that you typically 
> {{select()}} after {{match()}}, a {{Map<String,Object>}} is generated. The 
> benefit of making it a {{FilterStep}} is that people can use {{match()}} to 
> filter without having to deal with unrolling the {{Map}}. Moreover, it feels 
> more consistent with the language constructs.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to