Re: Review Request 38315: Change ZooKeeperTestServer to not rely on ShutdownRegistry.

2015-09-11 Thread Bill Farner
> On Sept. 11, 2015, 1:52 p.m., Zameer Manji wrote: > > commons/src/main/java/org/apache/aurora/common/application/AppLauncher.java, > > line 96 > > > > > > Shouldn't we log this instead of printing it? Doh, this

Re: Review Request 38315: Change ZooKeeperTestServer to not rely on ShutdownRegistry.

2015-09-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38315/ --- (Updated Sept. 11, 2015, 2:12 p.m.) Review request for Aurora and Zameer

Re: Review Request 38315: Change ZooKeeperTestServer to not rely on ShutdownRegistry.

2015-09-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38315/ --- (Updated Sept. 11, 2015, 3:28 p.m.) Review request for Aurora and Zameer

Re: Review Request 38315: Change ZooKeeperTestServer to not rely on ShutdownRegistry.

2015-09-11 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38315/#review98707 --- Ship it! Master (ec02a49) is green with this patch.