GitHub user davies opened a pull request:
https://github.com/apache/spark/pull/8286
[SPARK-10095] [SQL] use public API of BigInteger
In UnsafeRow, we use the private field of BigInteger for better
performance, but it actually didn't contribute much (3% in one benchmark) to
end-to-end runtime, and make it not portable (may fail on other JVM
implementations).
So we should use the public API instead.
cc @rxin
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/davies/spark portable_decimal
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/8286.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 #8286
----
commit e547fe80f59f83fe2b3934215975f9180c5da164
Author: Davies Liu <dav...@databricks.com>
Date: 2015-08-18T20:59:58Z
use public API of BigInteger
----
---
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.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
