[ 
https://issues.apache.org/jira/browse/TINKERPOP-1180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15190815#comment-15190815
 ] 

ASF GitHub Bot commented on TINKERPOP-1180:
-------------------------------------------

Github user spmallette commented on the pull request:

    
https://github.com/apache/incubator-tinkerpop/pull/254#issuecomment-195324592
  
    VOTE +1


> Add more optimized binary operators to Operator.
> ------------------------------------------------
>
>                 Key: TINKERPOP-1180
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1180
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.1.1-incubating
>            Reporter: Marko A. Rodriguez
>
> We currently have {{Operator}} enum which has nice operators like {{add}}, 
> {{assign}}, {{sum}}, {{max}}, {{min}}, etc. The problem with lots of the 
> number-based implementations is that they reason about the number format 
> using {{NumberHelper}}. I think we should add {{sumLong}}, {{sumInteger}}, 
> and the like to allow classes the leverage {{Operator}} to bypass 
> number-type-reasoning when they know it will be a long, integer, etc.
> This is important for {{GraphComputer.Memory}} where reducers are binary 
> operators and we use {{Operator}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to