Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/6360#issuecomment-110863727
> the YARN code is getting quite difficult to review
Yeah, that comes from a long time. It really needs some cleaning up. It's
not just the size of methods, but the implicit dependencies they have on one
another (method A sets some instance field that is needed by method B, so they
must be executed in that order or things break). That's the trickiest thing
when messing with this code (the size is not that big of an issue IMO).
---
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]
