[GitHub] tinkerpop issue #734: TINKERPOP-1801: fix profile() timing in OLAP by adding...

2017-10-24 Thread artem-aliev
Github user artem-aliev commented on the issue: https://github.com/apache/tinkerpop/pull/734 Something strange happened, during rebases. I'll fix. ---

[GitHub] tinkerpop issue #734: TINKERPOP-1801: fix profile() timing in OLAP by adding...

2017-10-23 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/734 This doesn't build for me with just regular `mvn clean install` ---

[GitHub] tinkerpop issue #734: TINKERPOP-1801: fix profile() timing in OLAP by adding...

2017-10-22 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/734 VOTE: +1 ---

[GitHub] tinkerpop issue #734: TINKERPOP-1801: fix profile() timing in OLAP by adding...

2017-10-21 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/734 VOTE +1 ---

[GitHub] tinkerpop issue #734: TINKERPOP-1801: fix profile() timing in OLAP by adding...

2017-10-20 Thread artem-aliev
Github user artem-aliev commented on the issue: https://github.com/apache/tinkerpop/pull/734 The fix add iteration time to the appropriate step. Iteration time is a time between workerIterationStart() and workerIterationEnd() callbacks. So before fix the timing looks like ```

[GitHub] tinkerpop issue #734: TINKERPOP-1801: fix profile() timing in OLAP by adding...

2017-10-18 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/734 This is a nice update @artem-aliev because it doesn't change API and it is general for all `GraphComputer` implementations. Great! A couple things please for a solid VOTE. 1. Please update