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

Matt Frantz commented on TINKERPOP3-793:
----------------------------------------

If you want an exception, yes, the current implementation is sufficient.  I was 
trying to articulate that if you want to avoid the exception, i.e. the 
{{filter}} suggestion, it is awkward.

{noformat}
...filter(X).filter(Y).filter(Z)
  .select('x', 'y', 'z')
  .by(X).by(Y).by(Z)
{noformat}


> select should stop traversal on unproductive by
> -----------------------------------------------
>
>                 Key: TINKERPOP3-793
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP3-793
>             Project: TinkerPop 3
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.0.1-incubating
>            Reporter: Matt Frantz
>            Assignee: Marko A. Rodriguez
>              Labels: breaking
>             Fix For: 3.1.0-incubating
>
>
> Instead of throwing an exception, we should stop traversal if a {{by}} clause 
> fails to produce.
> Current:
> {noformat}
> gremlin> g.V().as('a').out().out().as('b')
>   .select('a', 'b')
>   .by('name')
>   .by(has('name', 'lop').values('name'))
> The provided start does not map to a value: v[5]->[HasStep([name.eq(lop)])]
> {noformat}
> Desired:
> {noformat}
> gremlin> g.V().as('a').out().out().as('b')
>   .select('a', 'b')
>   .by('name')
>   .by(has('name', 'lop').values('name'))
> ==>[a:marko, b:lop]
> {noformat}



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

Reply via email to