[GitHub] tinkerpop issue #450: Javascript GLV

2016-10-05 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/450 @jorgebay thanks for this. I have a feeling it will take a some time to review this work and get community input. hopefully @jbmusso will have time to get involved. in the mean time co

[GitHub] tinkerpop issue #450: Javascript GLV

2016-10-05 Thread jorgebay
Github user jorgebay commented on the issue: https://github.com/apache/tinkerpop/pull/450 Thanks @PommeVerte for getting a first look at it so fast. About `list()` and `one()`: - `next()` is a method exposed by the [Iterator protocol][1] which in newer versions of Ecm

[GitHub] tinkerpop issue #450: Javascript GLV

2016-10-05 Thread PommeVerte
Github user PommeVerte commented on the issue: https://github.com/apache/tinkerpop/pull/450 Nice PR @jorgebay Can you elaborate on why you went for `list()` and `one()` rather than keeping the original method names of `toList()` and `next()`? I understand that they would still ha