Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/9084#discussion_r75595837
--- Diff: core/src/main/scala/org/apache/spark/memory/MemoryManager.scala
---
@@ -40,6 +41,10 @@ private[spark] abstract class MemoryManager {
_memoryStore
}
+ // Amount of execution/storage memory in use, accesses must be
synchronized on `this`
+ protected var _executionMemoryUsed: Long = 0
--- End diff --
@pzz2011 you're making several comments on old PRs. Generally people won't
see that and it's not the place for discussion anyway. If you can formulate a
specific question beyond "why is the code this way?" ask on user@.
---
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]
