Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2485
@revans2 I took another look after some of your last comments, and found
some more bugs. I spent some time splitting up NormalizedResources so it was
easier to test. Your comments should also have been addressed.
Most of the classes related to NormalizedResources have been moved to a new
package. The move is in the last commit, so it's probably easier to exclude
that one from the diff.
If you have an idea for a better place to put the static fields in
NormalizedResources, I'd be happy to move them. They are referred to from a lot
of different classes, and it would be better if we could create the classes
somewhere during startup and inject them into the classes that need them
instead of referring to these static fields.
---
