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 <[email protected]>
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 [email protected] or file a JIRA ticket
with INFRA.
---
