Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15218
@zhzhan in general it'd be great to have proper documentation on the
classes. For example, it is important to document the behavior of the various
assigners, and even more importantly, document the contract for TaskAssigner.
The control flow is fairly confusing right now -- I'm not very smart and things
that are complicated take me a long time to understand, and when I try changing
them in the future, there's a very good chance I will make a mistake and mess
it up. It would be great if we can simplify the control flow. If we can't, then
we should document it more clearly. For example, when init/reset should be
called are all part of the contracts, and none of them are really documented.
---
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]
