Github user miguno commented on the pull request:
https://github.com/apache/incubator-storm/pull/103#issuecomment-43896389
@clockfly Many thanks for your all the hard work and your patience. It's
much appreciated!
I don't have anything to add to the discussion at the moment (thanks Bobby
& Co.!) except that I, too, can confirm that the Storm test suite passes with
Sean's latest code changes. I tested against the latest commit in Sean's
`storm_async_netty_and_batch_api` branch, which at the time of writing was
https://github.com/clockfly/incubator-storm/commit/20b4f8b2195a1bf214f63e10b1bbca4690c0290f.
$ git checkout master
$ gco -b STORM-297
$ git pull [email protected]:clockfly/incubator-storm.git
storm_async_netty_and_batch_api
$ mvn clean install
>>> Success.
PS: Unfortunately I haven't had the chance yet to run the patched version
of Storm in a large-scale environment.
---
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.
---
