Github user aarondav commented on the pull request:
https://github.com/apache/spark/pull/3253#issuecomment-62966799
LGTM too. As @vanzin said, this may allow us to introduce code that breaks
only when run inside the NM, which is pretty bad. Until we can find a better
way to manage this issue, we'll just have to closely audit any changes made in
this package which touch Guava.
---
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]
