diff options
| author | Adam Roberts <aroberts@uk.ibm.com> | 2016-11-04 12:06:06 -0700 |
|---|---|---|
| committer | Reynold Xin <rxin@databricks.com> | 2016-11-04 12:06:06 -0700 |
| commit | a42d738c5de08bd395a7c220c487146173c6c163 (patch) | |
| tree | 1018e34a21f2145ad8a42ea6f58a876ce678dc2c /sql/catalyst | |
| parent | 14f235d5643bca75e270652c15154d86e57a7a70 (diff) | |
| download | spark-a42d738c5de08bd395a7c220c487146173c6c163.tar.gz spark-a42d738c5de08bd395a7c220c487146173c6c163.tar.bz2 spark-a42d738c5de08bd395a7c220c487146173c6c163.zip | |
[SPARK-18197][CORE] Optimise AppendOnlyMap implementation
## What changes were proposed in this pull request?
This improvement works by using the fastest comparison test first and we observed a 1% throughput performance improvement on PageRank (HiBench large profile) with this change.
We used tprof and before the change in AppendOnlyMap.changeValue (where the optimisation occurs) this method was being used for 8053 profiling ticks representing 0.72% of the overall application time.
After this change we observed this method only occurring for 2786 ticks and for 0.25% of the overall time.
## How was this patch tested?
Existing unit tests and for performance we used HiBench large, profiling with tprof and IBM Healthcenter.
Author: Adam Roberts <aroberts@uk.ibm.com>
Closes #15714 from a-roberts/patch-9.
Diffstat (limited to 'sql/catalyst')
0 files changed, 0 insertions, 0 deletions