Github user d2r commented on the pull request:
https://github.com/apache/storm/pull/921#issuecomment-162723108
Regarding the eviction strategy diagram:
* One of the bubbles it out-sized by the text inside it.
* The text size is pretty small. Can we use an SVG or a larger resolution
instead?
* In fact, I cannot tell if the diagram is correct, because I cannot see
clearly enough the difference between the `i` subscript and the `j` subscript.
* We are missing the actual eviction, which should happen in the case we
identify topologies that should be killed. In the remaining case when the new
topology can be scheduled immediately, there is no eviction.
This is a really good document to have, so I wanted to make sure it is easy
enough to reference.
---
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.
---
