GitHub user jianran opened a pull request:
https://github.com/apache/spark/pull/17313
[SPARK-19974][Block Manager] in-memory LRU for partitions of multiple RDDs
testcase did a get that is no sense
## What changes were proposed in this pull request?
in-memory LRU for partitions of multiple RDDs testcase did a get that is no
sense; rdd_0_2 is the most recently used item that isn't the reason the
rdd_0_2 is remain; `store.getSingleAndReleaseLock(rdd(0, 2)).isDefined ` as
same as the previous line `store.memoryStore.contains(rdd(0, 2))`, so remove
the no sense code
## How was this patch tested?
run the test
(Please explain how this patch was tested. E.g. unit tests, integration
tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise,
remove this)
Please review http://spark.apache.org/contributing.html before opening a
pull request.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jianran/spark samerddrule
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/17313.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 #17313
commit 2710de064ee3cad8c5db117f8fd4b1a984303b1e
Author: jianran.tfh
Date: 2017-03-16T07:05:28Z
same rdd rule testcase
commit a740052594352f45267812fc65a04082dcf3913c
Author: jianran.tfh
Date: 2017-03-16T10:07:17Z
rollback
commit b3b97086bbb5e4415e5694fcc0cd33f4511645a9
Author: jianran.tfh
Date: 2017-03-16T10:17:30Z
remove get
---
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