[
https://issues.apache.org/jira/browse/TINKERPOP3-941?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15023585#comment-15023585
]
ASF GitHub Bot commented on TINKERPOP3-941:
-------------------------------------------
Github user twilmes commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/155#issuecomment-159129937
D'argh, party foul. Got it, thanks.
VOTE +1
> Improve error message for wrong order().by() arguments
> ------------------------------------------------------
>
> Key: TINKERPOP3-941
> URL: https://issues.apache.org/jira/browse/TINKERPOP3-941
> Project: TinkerPop 3
> Issue Type: Improvement
> Components: process
> Affects Versions: 3.0.2-incubating
> Reporter: Daniel Kuppitz
> Assignee: Marko A. Rodriguez
> Fix For: 3.1.1-incubating
>
>
> The traversal {code}g.V(hercules).local(__.outE().order().by("lala")){code}
> throws the following exception:
> {noformat}
> java.lang.UnsupportedOperationException: Use
> OrderGlobalStep.addComparator(TraversalComparator) to add a local child
> traversal:OrderGlobalStep
> at
> org.apache.tinkerpop.gremlin.process.traversal.step.map.OrderGlobalStep.addLocalChild(OrderGlobalStep.java:98)
> at
> org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversal.by(GraphTraversal.java:1016)
> at
> org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversal.by(GraphTraversal.java:1033)`
> {noformat}
> If you ran into this exception a few times, you'll know how easy it is to fix
> that, but I concur with [~mbroecheler] when he says that this message is
> really confusing. Users won't realize that they need to add {{incr}} or
> {{decr}} as a second argument in {{by()}}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
