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 <jianran....@taobao.com> Date: 2017-03-16T07:05:28Z same rdd rule testcase commit a740052594352f45267812fc65a04082dcf3913c Author: jianran.tfh <jianran....@taobao.com> Date: 2017-03-16T10:07:17Z rollback commit b3b97086bbb5e4415e5694fcc0cd33f4511645a9 Author: jianran.tfh <jianran....@taobao.com> 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