[GitHub] tinkerpop issue #852: TINKERPOP-1944 Export DriverRemoteConnection

2018-04-26 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/852 VOTE +1 ---

[GitHub] tinkerpop issue #695: TINKERPOP-1489 JavaScript GLV

2018-01-23 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/695 All good to me! VOTE +1 ---

[GitHub] tinkerpop issue #695: TINKERPOP-1489 JavaScript GLV

2018-01-19 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/695 According to https://github.com/petkaantonov/bluebird/issues/1026, users should be able to just patch the global `Promise` object in their application with: ```javascript global.Promise

[GitHub] tinkerpop issue #695: TINKERPOP-1489 JavaScript GLV

2018-01-17 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/695 Finally found some time. Wew. Well, this PR is very well crafted - well done! My comments are only minor: * About `utils.toPromise` - if I understand you well, you want a dual

[GitHub] tinkerpop issue #695: TINKERPOP-1489 JavaScript GLV

2018-01-11 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/695 Quick update - I plan to check this PR this weekend. ---

[GitHub] tinkerpop issue #450: TINKERPOP-1489 Javascript GLV

2017-06-02 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/450 @jorgebay I'd be happy to help with the work on this. I recently [added partial JavaScript-GLV support to gremlin-javascript](https://github.com/jbmusso/gremlin-javascript

[GitHub] tinkerpop issue #450: TINKERPOP-1489 Javascript GLV

2016-11-10 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/450 Ah, it looks like [nashorn supports let and const keywords on 8u40](http://fr.slideshare.net/akihironishikawa/nashorn-in-the-future-en) so we could swap `var` for `let` and `const` unless we want

[GitHub] tinkerpop issue #450: TINKERPOP-1489 Javascript GLV

2016-10-08 Thread jbmusso
Github user jbmusso commented on the issue: https://github.com/apache/tinkerpop/pull/450 I was thinking that supporting JavaScript `Promise` could be helpful now that they’ve been part of the ES2015 and rolled out on most JS environment (browsers/Node.js). Maybe this can be