GitHub user suyanNone reopened a pull request:

    https://github.com/apache/spark/pull/3629

    [SPARK-4777][CORE] Some block memory after unrollSafely not count into used 
memory(memoryStore.entrys or unrollMemory)

    Some memory not count into memory used by memoryStore or unrollMemory.
    Thread A after unrollsafely memory, it will release 40MB unrollMemory(40MB 
will used by other threads). then ThreadA wait get accountingLock to tryToPut 
blockA(30MB). before Thread A get accountingLock, blockA memory size is not 
counting into unrollMemory or memoryStore.currentMemory.
    IIUC, freeMemory should minus that block memory
    
    So, put this release memory into pending, and release it in tryToPut before 
ensureSpace 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/suyanNone/spark unroll-memory

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/3629.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 #3629
    
----
commit 072e43d49226f1ae660d9b2ad53dc43ee78481e9
Author: hushan[胡珊] <[email protected]>
Date:   2014-12-05T02:56:20Z

    Pending unroll memory for this block untill tryToPut

commit 4f46dd7dc985d6e9eeb907f3f3e9ab010fa2f27d
Author: hushan[胡珊] <[email protected]>
Date:   2014-12-08T07:15:58Z

    Make sure not add pending in every releaseUnrollMemory call

commit 42dfa9ba9b092ad935740d9ca75a0a8166474d62
Author: hushan[胡珊] <[email protected]>
Date:   2014-12-11T02:28:37Z

    Refine getOrElse

commit 34cfbe8b309addb98deb23429626b14cb13a8e2a
Author: hushan[胡珊] <[email protected]>
Date:   2014-12-11T07:08:41Z

    Refine blockManagerSuite unroll test

----


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

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

Reply via email to