SuYan created SPARK-6156:
----------------------------

             Summary: Refine Put Memory_And_Disk block
                 Key: SPARK-6156
                 URL: https://issues.apache.org/jira/browse/SPARK-6156
             Project: Spark
          Issue Type: Bug
            Reporter: SuYan


Current code:
Now we want to cache a Memory_and_Disk level block
1. Try to put in memory and unroll unsuccessful. then reserved unroll memory 
because we got a iterator from an unroll Array 
2. Then put into disk.
3. Get value from get(blockId), and iterator from that value.

And there have some doubts:
for process 3:

get value use blockManager.get(blockId)
in get(blockId) method, it will first getLocal() and then getRemote() in 
logical, I think is more suitable for use getLocal(blockId) to instead 
get(blockId), I dont know why we call get(blockId), have other considerations?

and in doGetLocal method, if it get value success from disk, it will try to put 
in memory again... I think, that a redundant... we just try to put in memory 
and failed to unroll in process 1.

may this pull request is useless, because may have some reasons to do that?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to