Review Request 40510: Remove unused and deprecated code from commons.

2015-11-19 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40510/ --- Review request for Aurora, John Sirois and Bill Farner. Repository: aurora

Re: Review Request 40510: Remove unused and deprecated code from commons.

2015-11-19 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40510/ --- (Updated Nov. 19, 2015, 4:13 p.m.) Review request for Aurora, John Sirois and

Re: Review Request 40510: Remove unused and deprecated code from commons.

2015-11-19 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40510/ --- (Updated Nov. 19, 2015, 4:25 p.m.) Review request for Aurora, John Sirois and

Re: Review Request 40510: Remove unused and deprecated code from commons.

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

Re: Review Request 30695: Implements log rotation in the Thermos runner.

2015-11-19 Thread Stephan Erb
> On Feb. 6, 2015, 7:52 p.m., Brian Wickman wrote: > > This is super rad. Thanks for taking this on. > > > > Before I do a deeper dive, what do you think about making the logrotate > > policy be specified by the user instead of the framework owner, with a > > sensible default? For example,

Re: Review Request 40466: Kill unused `.st` files and params.

2015-11-19 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40466/#review107193 --- Ship it! Ship It! - Joshua Cohen On Nov. 19, 2015, 3:46 a.m.,