Re: Review Request 37872: Remove unused classes from commons fork.

2015-08-28 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37872/#review96898 --- Ship it! Master (06ddaad) is green with this patch. ./build-suppo

Re: Review Request 37872: Remove unused classes from commons fork.

2015-08-28 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37872/ --- (Updated Aug. 28, 2015, 10:15 a.m.) Review request for Aurora and Zameer Manji.

Re: Review Request 37872: Remove unused classes from commons fork.

2015-08-28 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37872/#review96889 --- Ship it! Ship It! - Zameer Manji On Aug. 27, 2015, 9:46 p.m., Bi

Re: Review Request 37872: Remove unused classes from commons fork.

2015-08-28 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37872/#review96886 --- Since you deleted commons/src/main/java/org/apache/aurora/common/te

Re: Review Request 37872: Remove unused classes from commons fork.

2015-08-27 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37872/#review96834 --- Ship it! Master (06ddaad) is green with this patch. ./build-suppo