Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/11874#issuecomment-200594484
I left a few suggestions on how to rewrite things in a way that's more
readable. I'm not sure if we should change `StaticMemoryManager` since new
users will probably not use that, and old users already suffer from the lock
contention problem. Making changes there might introduce regressions and
increase the surface area of things we need to maintain in Spark 2.0.
---
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]
