You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by GitBox <gi...@apache.org> on 2020/11/11 15:56:36 UTC

[GitHub] [tinkerpop] spmallette opened a new pull request #1356: TINKERPOP-2001 Add lambda support in gremlin-javascript

spmallette opened a new pull request #1356:
URL: https://github.com/apache/tinkerpop/pull/1356


   https://issues.apache.org/jira/browse/TINKERPOP-2001
   
   Pretty straightforward - follows the same pattern as python basically. No need to support native javascript lambdas at this time (anymore than there is a future need to support native python lambdas at this time) as there is no legitimate execution engine for them (though that would be neat as you could write much more fluent Gremlin in javascript that way). 
   
   This change enabled almost 30 tests that were just being ignored and paves the way for us to write more advanced GLV tests for other purposes. 
   
   Builds with `mvn clean install`:
   
   VOTE +1


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tinkerpop] jorgebay commented on pull request #1356: TINKERPOP-2001 Add lambda support in gremlin-javascript

Posted by GitBox <gi...@apache.org>.
jorgebay commented on pull request #1356:
URL: https://github.com/apache/tinkerpop/pull/1356#issuecomment-725513067


   WOOT! nice!
   
   I'll review it shortly.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tinkerpop] spmallette merged pull request #1356: TINKERPOP-2001 Add lambda support in gremlin-javascript

Posted by GitBox <gi...@apache.org>.
spmallette merged pull request #1356:
URL: https://github.com/apache/tinkerpop/pull/1356


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org