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

Reply via email to