I appreciate it. I know it is a lot of code to go though. If others are planning to take a look, even a small one, please let me know. I really do want to get this merged in, because constantly updating it will be a pain, but I also want to give everyone who wants one a chance to find mistakes that I have made. I don't think that it is possible for me to have written that many lines of code with no uncaught mistakes in it, even with someone like Taylor reading through all of it.
- Bobby On Tuesday, November 22, 2016, 2:44:22 PM CST, P. Taylor Goetz <ptgo...@gmail.com> wrote:I got one review done (Nimbus). Second one will likely have to wait until after the holiday as I will be out. Nice work and thanks for pushing this forward. -Taylor > On Nov 21, 2016, at 10:46 AM, Bobby Evans <ev...@yahoo-inc.com.INVALID> wrote: > > I have been pushing hard to get some of the last code moved over to java. > I currently have a pull request up for nimbus > https://github.com/apache/storm/pull/1744 > and another one based off of the nimbus pull request to move LocalCluster and > a lot of the other testing helper classes to java. > https://github.com/apache/storm/pull/1786 > I realize that this is kind of daunting. In aggregate it is over 8,000 lines > added and about 6,500 lines deleted. But any help people can give with > reviews would really be appreciated. > After these go in what is left is the ui, logviewer, a tiny amount in DRPC, > and the IsolationScheduler (which has a pull request up) for the main daemons > and then several tests. > Thanks, > Bobby
