Review Request 38210: [aurora-packaging] Adds Monit supervisor support for EL6

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38210/ --- Review request for Aurora, Kevin Sweeney and Bill Farner. Bugs: AURORA-1480

Re: Review Request 38200: Adding notes on changing the scheduler quorum size

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38200/#review98175 --- Ship it! Master (2773826) is green with this patch.

Re: Review Request 38200: Adding notes on changing the scheduler quorum size

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38200/ --- (Updated Sept. 9, 2015, 1:36 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 38200: Adding notes on changing the scheduler quorum size

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38200/#review98189 --- Ship it! I'll go ahead and commit this once the cleanups are

Re: Review Request 38200: Adding notes on changing the scheduler quorum size

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38200/#review98201 --- Ship it! Master (2773826) is green with this patch.

Re: Review Request 38202: Replace BoundedQueue with Guava's EvictingQueue.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38202/#review98286 --- Ship it! Ship It! - Bill Farner On Sept. 8, 2015, 8:33 p.m.,

Re: Review Request 38014: Remove StartupRegistry.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38014/#review98297 --- Ship it! Master (a5ac5b9) is green with this patch.

Re: Review Request 38014: Remove StartupRegistry.

> On Sept. 9, 2015, 2:39 p.m., Kevin Sweeney wrote: > > commons/src/main/java/org/apache/aurora/common/application/AppLauncher.java, > > line 63 > > > > > > Looks like Modules.combine is redundant here. Thanks,

Re: Review Request 38086: Add test environments for RPMs and debs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38086/ --- (Updated Sept. 9, 2015, 3:19 p.m.) Review request for Aurora and Kevin

Re: Review Request 38086: Add test environments for RPMs and debs.

> On Sept. 9, 2015, 11 a.m., Kevin Sweeney wrote: > > test/deb/ubuntu-trusty/Vagrantfile, lines 6-12 > > > > > > mutli-vm boilerplate isn't needed > > > > Can replace this block with > > > > ```ruby

Re: Review Request 38202: Replace BoundedQueue with Guava's EvictingQueue.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38202/#review98302 --- Ship it! Master (a5ac5b9) is green with this patch.

Re: Review Request 38112: Alter thrift wrapper generator to use default primitive values and empty collections.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38112/#review98345 --- Maxim - ping. - Bill Farner On Sept. 8, 2015, 4:26 p.m., Bill

Re: Review Request 38081: Dropping quota check for dedicated jobs and exposing dedicated consumption.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38081/ --- (Updated Sept. 9, 2015, 6:35 p.m.) Review request for Aurora, Joshua Cohen,

Re: Review Request 38081: Dropping quota check for dedicated jobs and exposing dedicated consumption.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38081/#review98238 --- Ship it! Master (4577de4) is green with this patch.

Re: Review Request 38169: Refactoring resources table to show dedicated.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38169/#review98260 --- Master (01e611a) is green with this patch.

Re: Review Request 38169: Refactoring resources table to show dedicated.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38169/#review98268 --- Ship it! Ship It! - David McLaughlin On Sept. 9, 2015, 7:10

Re: Review Request 38086: Add test environments for RPMs and debs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38086/#review98272 --- Ship it! LGTM once comments are addressed - Kevin Sweeney On

Re: Review Request 38014: Remove StartupRegistry.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38014/#review98274 --- Ship it!