[GitHub] tinkerpop pull request #711: TINKERPOP-1746: Better error message on wrong o...

[dkuppitz]

Github user dkuppitz commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/711#discussion_r138436841
  
    --- Diff: 
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/branch/RepeatStep.java
 ---
    @@ -91,6 +91,8 @@ public void setEmitTraversal(final Traversal.Admin<S, ?> 
emitTraversal) {
         }
     
         public Traversal.Admin<S, S> getRepeatTraversal() {
    +        if(null == this.repeatTraversal)
    +            throw new IllegalStateException("The repeat()-traversal was 
not defined: " + this);
    --- End diff --
    
    I wouldn't throw an exception in a Getter.


---