[GitHub] tinkerpop issue #711: TINKERPOP-1746: Better error message on wrong ordering...

2017-09-18 Thread spmallette 
Github user spmallette commented on the issue:

https://github.com/apache/tinkerpop/pull/711
  
All tests pass with `docker/build.sh -t -n -i`

VOTE +1


---

[GitHub] tinkerpop issue #711: TINKERPOP-1746: Better error message on wrong ordering...

2017-09-13 Thread dkuppitz 
Github user dkuppitz commented on the issue:

https://github.com/apache/tinkerpop/pull/711
  
Nvm, I just saw the ticket description (link is missing here and I was too 
lazy to look it up).

VOTE: +1


---

[GitHub] tinkerpop issue #711: TINKERPOP-1746: Better error message on wrong ordering...

2017-09-13 Thread dkuppitz 
Github user dkuppitz commented on the issue:

https://github.com/apache/tinkerpop/pull/711
  
Which scenario would end up throwing this exception? Tried this:

```
gremlin> g = TinkerFactory.createModern().traversal()
==>graphtraversalsource[tinkergraph[vertices:6 edges:6], standard]
gremlin> g.V().repeat().times(2)
java.lang.NullPointerException
```

... but it's obviously still a NPE thrown in [line 
67](https://github.com/apache/tinkerpop/blob/53db262511e9d84b1262522fc841ded825df4fc6/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/branch/RepeatStep.java#L67).


---