Re: Review Request 32302: Allow config to specify static ports for non-dedicated jobs.

2015-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32302/#review107136 --- This patch does not apply cleanly on master (7b43b99), do you need

Re: Review Request 32302: Allow config to specify static ports for non-dedicated jobs.

2015-11-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32302/#review107133 --- Brian - can you add specific reviewers to the `People` field or els

Re: Review Request 37851: Use "Mesos Agent" instead of "Mesos Slave" in docs.

2015-11-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37851/#review107132 --- Ship it! Ship It! - John Sirois On Oct. 9, 2015, 10:38 a.m., Ke

Re: Review Request 37851: Use "Mesos Agent" instead of "Mesos Slave" in docs.

2015-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37851/#review107131 --- Ship it! Master (6ba06ae) is green with this patch. ./build-supp

Re: Review Request 36874: Fix typo in the scheduler deployment documentation

2015-11-18 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36874/#review107130 --- Ship it! This is now on master: ``` $ git log -1 origin/master com

Re: Review Request 36874: Fix typo in the scheduler deployment documentation

2015-11-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36874/#review107129 --- Ship it! Ping - This should be patched in by a committer FWICT. -

Re: Review Request 36289: Custom executor support for Scheduler

2015-11-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36289/#review107128 --- This has been idle a good while and it appears wfarner has picked u

Re: Review Request 37851: Use "Mesos Agent" instead of "Mesos Slave" in docs.

2015-11-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37851/#review107127 --- @ReviewBot retry - John Sirois On Oct. 9, 2015, 10:38 a.m., Kevi

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

2015-11-18 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40466/#review107124 --- Ship it! Ship It! - Bill Farner On Nov. 18, 2015, 7:46 p.m., Jo

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

2015-11-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40466/ --- (Updated Nov. 18, 2015, 8:46 p.m.) Review request for Aurora, Joshua Cohen, Max

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

2015-11-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40466/#review107118 --- Master (7b1e848) is green with this patch. ./build-support/jenkin

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

2015-11-18 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40466/ --- Review request for Aurora, Joshua Cohen, Maxim Khutornenko, and Bill Farner. Re