Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/1642
@knusbaum @srdo @harshach @HeartSaVioR @abellina I really would appreciate
it if you could take another look at this patch. I think everything is ready
to go except a final squash before merging it. The unit tests all pass, I have
run some basic manual tests with CGroups and run as user enabled. I have run a
number of manual tests on my mac, including rebalancing, killing the workers,
killing the supervisor, and going back and forth between old and new
supervisor. I have manually deleted and added files/directories in storm-dist
on the supervisor. I also ran the full set of the internal yahoo integration
tests against it and everything has passed.
I really would like to start rolling this out internally at yahoo, but I
want to get as many eyes looking at it to find issues first. Better here then
in production. On the upside the bugs I have found have been rather easy to
debug, and very reproducible because the race conditions have been eliminated
when interacting with the Container.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
