[ https://issues.apache.org/jira/browse/TINKERPOP-1746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16161689#comment-16161689 ]
ASF GitHub Bot commented on TINKERPOP-1746: ------------------------------------------- GitHub user okram opened a pull request: https://github.com/apache/tinkerpop/pull/711 TINKERPOP-1746: Better error message on wrong ordering of emit()/until()/has() Simply added a proper error message when `repeatTraversal` is null. This provides a better user experience than `NullPointerException`. VOTE +1 You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/tinkerpop TINKERPOP-1746 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tinkerpop/pull/711.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #711 ---- commit b28701f7c660c87b05edee3404a66c2fb3671089 Author: Marko A. Rodriguez <okramma...@gmail.com> Date: 2017-09-11T17:56:20Z added better error message for RepeatStep usage errors. ---- > Better error message on wrong ordering of emit()/until()/has() > -------------------------------------------------------------- > > Key: TINKERPOP-1746 > URL: https://issues.apache.org/jira/browse/TINKERPOP-1746 > Project: TinkerPop > Issue Type: Improvement > Components: process > Affects Versions: 3.2.5 > Environment: TinkerPop 3.2.5 > Reporter: DOAN DuyHai > Priority: Trivial > > {code} > gremlin> g.withSack([] as Set). > ......1> V(). > ......2> emit(). > ......3> has("station", "name", "South Kensington"). > ......4> repeat(bothE("connectedTo").sack{s,v -> s.add(v); return > s;}.by("line").otherV().simplePath()).times(2). > ......5> sack() > NullPointerException > Type ':help' or ':h' for help. > Display stack trace? [yN] > {code} -- This message was sent by Atlassian JIRA (v6.4.14#64029)