diff options
| author | Liang-Chi Hsieh <viirya@gmail.com> | 2016-02-03 23:17:51 -0800 |
|---|---|---|
| committer | Davies Liu <davies.liu@gmail.com> | 2016-02-03 23:17:51 -0800 |
| commit | d39087147ff1052b623cdba69ffbde28b266745f (patch) | |
| tree | 179bf53e7a238b27dad8ec9b2793b1d34a379a53 | |
| parent | c2c956bcd1a75fd01868ee9ad2939a6d3de52bc2 (diff) | |
| download | spark-d39087147ff1052b623cdba69ffbde28b266745f.tar.gz spark-d39087147ff1052b623cdba69ffbde28b266745f.tar.bz2 spark-d39087147ff1052b623cdba69ffbde28b266745f.zip | |
[SPARK-13113] [CORE] Remove unnecessary bit operation when decoding page number
JIRA: https://issues.apache.org/jira/browse/SPARK-13113
As we shift bits right, looks like the bitwise AND operation is unnecessary.
Author: Liang-Chi Hsieh <viirya@gmail.com>
Closes #11002 from viirya/improve-decodepagenumber.
| -rw-r--r-- | core/src/main/java/org/apache/spark/memory/TaskMemoryManager.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/src/main/java/org/apache/spark/memory/TaskMemoryManager.java b/core/src/main/java/org/apache/spark/memory/TaskMemoryManager.java index d31eb449eb..d2a88864f7 100644 --- a/core/src/main/java/org/apache/spark/memory/TaskMemoryManager.java +++ b/core/src/main/java/org/apache/spark/memory/TaskMemoryManager.java @@ -312,7 +312,7 @@ public class TaskMemoryManager { @VisibleForTesting public static int decodePageNumber(long pagePlusOffsetAddress) { - return (int) ((pagePlusOffsetAddress & MASK_LONG_UPPER_13_BITS) >>> OFFSET_BITS); + return (int) (pagePlusOffsetAddress >>> OFFSET_BITS); } private static long decodeOffset(long pagePlusOffsetAddress) { |