GitHub user okram opened a pull request:

    https://github.com/apache/incubator-tinkerpop/pull/254

    TINKERPOP-1180: Add more optimized binary operators to Operator

    https://issues.apache.org/jira/browse/TINKERPOP-1180
    
    This is simple. Added `Operator.sumLong` which bypasses using 
`NumberHelper`. Smart when you know you are summing longs as in 
`CountGlobalStep`. I also tweaked the initial capacity of the singleton maps of 
the `GroupXXXSteps`. As we need more Operators, we can just CTR add them. Its 
trivial.
    
    `mvn clean install`.
    
    VOTE +1.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/incubator-tinkerpop TINKERPOP-1180

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-tinkerpop/pull/254.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #254
    
----
commit 0d79c2ff7cb89a0fd81f2a2aa555d6d024a6de70
Author: Marko A. Rodriguez <okramma...@gmail.com>
Date:   2016-03-09T16:22:44Z

    Added Operator.sumLong for use by CountGlobalStep. I couldn't get 
SingletonMap to work for the GroupXXXSteps. Instead, just made the HashMap they 
generate have an initial capacity of 1.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to