Re: Review Request 33869: Fix inconsistency in MemTaskStore secondary indices.

2015-05-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33869/#review82614 --- Ship it! Master (751d65f) is green with this patch. ./build-suppo

Review Request 33869: Fix inconsistency in MemTaskStore secondary indices.

2015-05-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33869/ --- Review request for Aurora and Maxim Khutornenko. Bugs: AURORA-1305 https://

Re: Review Request 33608: Added a status update throughput benchmark.

2015-05-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33608/#review82598 --- Master (d3b6e13) is green with this patch. ./build-support/jenkins

Re: Review Request 33854: Fix and enhance RC verification script.

2015-05-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33854/#review82595 --- Ship it! Master (d3b6e13) is green with this patch. ./build-suppo

Re: Review Request 33608: Added a status update throughput benchmark.

2015-05-05 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33608/ --- (Updated May 5, 2015, 10:24 p.m.) Review request for Aurora and Maxim Khutornen

Re: Review Request 33854: Fix and enhance RC verification script.

2015-05-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33854/ --- (Updated May 5, 2015, 10:14 p.m.) Review request for Aurora and Jake Farrell.

Re: Review Request 33608: Added a status update throughput benchmark.

2015-05-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33608/#review82591 --- Ship it! src/jmh/java/org/apache/aurora/benchmark/StatusUpdateBenc

Re: Review Request 33784: Update upstart configurations along with sources.

2015-05-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33784/ --- (Updated May 5, 2015, 7:48 p.m.) Review request for Aurora and Kevin Sweeney.

Re: Review Request 33854: Fix RC verification script.

2015-05-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33854/#review82561 --- Ship it! Master (22871c0) is green with this patch. ./build-suppo

Re: Review Request 33854: Fix RC verification script.

2015-05-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33854/ --- (Updated May 5, 2015, 6:49 p.m.) Review request for Aurora and Jake Farrell.

Re: Review Request 33612: Add a task store implementation that uses a relational database.

2015-05-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33612/#review82558 --- Ship it! Master (8a1c8bb) is green with this patch. ./build-suppo

Re: Review Request 33787: fix link to contributing page

2015-05-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33787/#review82554 --- Ship it! Master (8a1c8bb) is green with this patch. ./build-suppo

Re: Review Request 33612: Add a task store implementation that uses a relational database.

2015-05-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33612/ --- (Updated May 5, 2015, 6:21 p.m.) Review request for Aurora, Joshua Cohen, Kevin

Re: Review Request 33612: Add a task store implementation that uses a relational database.

2015-05-05 Thread Bill Farner
> On May 3, 2015, 2:34 p.m., Stephan Erb wrote: > > src/main/java/org/apache/aurora/scheduler/storage/db/DbTaskStore.java, line > > 147 > > > > > > Copy paste error. Should be `db_storage_save_tasks`. Good catch! Fi

Re: Review Request 33783: Add stats for update state transitions.

2015-05-05 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33783/#review82550 --- Ship it! Ship It! - David McLaughlin On May 2, 2015, 6:41 p.m.,

Re: Review Request 33784: Symlink upstart configurations in vagrant rather than copying.

2015-05-05 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33784/#review82549 --- Ship it! Ship It! - Kevin Sweeney On May 2, 2015, 11:58 a.m., Bi

Re: Review Request 33819: Make aurora-scheduler-kerberos.conf startup manual.

2015-05-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33819/#review82548 --- Ship it! LGTM pending e2e test result. - Bill Farner On May 4, 2

Re: Review Request 33787: fix link to contributing page

2015-05-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33787/#review82546 --- Ship it! Ship It! - Bill Farner On May 5, 2015, 6:01 p.m., Bhuva

Review Request 33854: Fix RC verification script.

2015-05-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33854/ --- Review request for Aurora and Jake Farrell. Repository: aurora Description --

Re: Review Request 33787: fix link to contributing page

2015-05-05 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33787/#review82543 --- Ship it! Ship It! - Zameer Manji On May 5, 2015, 11:01 a.m., Bhu

Re: Review Request 33787: fix link to contributing page

2015-05-05 Thread Bhuvan Arumugam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33787/ --- (Updated May 5, 2015, 6:01 p.m.) Review request for Aurora, Bill Farner and Zam