Github user rxin commented on the issue:
    @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 or file a JIRA ticket
with INFRA.

To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to