[
https://issues.apache.org/jira/browse/TINKERPOP3-895?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14963483#comment-14963483
]
ASF GitHub Bot commented on TINKERPOP3-895:
-------------------------------------------
Github user spmallette commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/111#issuecomment-149252205
VOTE: +1
Perhaps you were going to add this after the merge, but don't forget the
JIRA issue to add for future removal of the deprecated classes.
> Use "as BinaryOperator" and remove GBinaryOperator
> --------------------------------------------------
>
> Key: TINKERPOP3-895
> URL: https://issues.apache.org/jira/browse/TINKERPOP3-895
> Project: TinkerPop 3
> Issue Type: Bug
> Components: process
> Affects Versions: 3.0.2-incubating
> Reporter: Marko A. Rodriguez
> Assignee: Marko A. Rodriguez
> Fix For: 3.1.0-incubating
>
>
> In Gremlin-Groovy we have function wrappers like {{GSupplier}},
> {{GFunction}}, {{GBinaryOperator}}, etc. which I use to wrap closures so that
> {{GraphTraversal}} methods know what the closure represents. I just learned
> you can do this:
> {code}
> sackSum = { a,b ->
> [a[0] + b[0],a[1] + b[1]]
> } as BinaryOperator
> {code}
> WOW! That is a huge saving in terms of code management.
> This is breaking if people are using {{GXXX}} for something, however, with
> {{as XXX}}, they shouldn't be using {{GXXX}}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
