[
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 <[email protected]>
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)
