ASF GitHub Bot commented on TINKERPOP-1854:

Github user spmallette commented on the issue:

    I'll let @jorgebay dig into the c# stuff here, but from my view this looks 
pretty good and it's nice to see this feature fully supported now. Great to 
know that all those feature tests pass.  It's too bad that this doesn't match 
java too well:
    > Unfortunately, it's not possible to use this new Lambda type for 
arguments in the traversal API
    but i guess there's not much we can do about that.  Perhaps, this 
limitations should be explained in the documentation for "The Lambda Solution"? 
From a documentation perspective, I think that (1) this is a big enough 
improvement that you should make note of it in the upgrade documentation and 
(2) this needs a changelog entry before you merge.
    Otherwise, all tests pass with `docker/build.sh -t -n -i`
    VOTE +1

> Support lambdas in Gremlin.Net
> ------------------------------
>                 Key: TINKERPOP-1854
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1854
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: dotnet
>    Affects Versions: 3.3.0, 3.2.6
>            Reporter: Florian Hockmann
>            Assignee: Florian Hockmann
>            Priority: Major
> Gremlin.Net should support lambdas. We already discussed this in [the pull 
> request for TINKERPOP-1752|https://github.com/apache/tinkerpop/pull/712]. 
> Here is what [~spmallette] said over there:
> {quote}
> Any reason we don't support lambdas? Even if .NET can't support them natively 
> for some reason wouldn't we minimally support the ability to pass a 
> python/groovy/etc lambda? it's kinda weird that way, but i think back to the 
> point that kuppitz made on the dev list the other day where he stated that he 
> doesn't always find a way out of using lambdas in production systems he works 
> on - so ultimately users will need that kind of capability i think.
> {quote}
> C# lambdas would require some kind of C# parser on the server side, so at 
> least in the beginning a way to send lambdas from already supported languages 
> in Gremlin.Net should be enough.

This message was sent by Atlassian JIRA

Reply via email to