Re: Review Request 32161: Create a packer definition to build a base vagrant box.

2015-04-09 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32161/#review79577 --- Vagrantfile

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/#review79566 --- Master (f48d0e1) is green with this patch. ./build-support/jenkins

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-09 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/#review79562 --- Ship it! Ship It! - Bill Farner On April 9, 2015, 7:15 p.m., Max

Re: Review Request 33029: Use installDist instead of installApp.

2015-04-09 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33029/#review79561 --- Ship it! LGTM so long as end-to-end tests still pass. - Bill Farne

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-09 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/ --- (Updated April 9, 2015, 7:15 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 33029: Use installDist instead of installApp.

2015-04-09 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33029/#review79557 --- Ship it! Ship It! - Zameer Manji On April 9, 2015, 12:02 p.m., K

Review Request 33029: Use installDist instead of installApp.

2015-04-09 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33029/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description --

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/#review79556 --- Master (7e3e7c9) is red with this patch. ./build-support/jenkins/b

Re: Review Request 32993: Revert "Bump pants version to 0.0.32."

2015-04-09 Thread Zameer Manji
> On April 8, 2015, 7:22 p.m., Bill Farner wrote: > > Can you please file a ticket with your findings? As it stands, there's > > likely to be another fateful victim in the future. https://issues.apache.org/jira/browse/AURORA-1267 - Zameer ---

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-09 Thread Maxim Khutornenko
> On April 9, 2015, 2:29 a.m., Bill Farner wrote: > > Actually - can you add a benchmark to ThriftApiBenchmarks and include > > results in the description? Should be pretty easy to prove the improvement > > of this. Done. - Maxim ---

Re: Review Request 32559: Add Kerberos support to the scheduler

2015-04-09 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32559/ --- (Updated April 9, 2015, 11:44 a.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-09 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/ --- (Updated April 9, 2015, 6:43 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 33026: Revert "Make health check configurable"

2015-04-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33026/#review79542 --- Master (6384710) is red with this patch. ./build-support/jenkins/b

Re: Review Request 33026: Revert "Make health check configurable"

2015-04-09 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33026/#review79541 --- Ship it! Ship It! - Joshua Cohen On April 9, 2015, 6:03 p.m., Bi

Re: Review Request 33026: Revert "Make health check configurable"

2015-04-09 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33026/#review79540 --- Ship it! Ship It! - Kevin Sweeney On April 9, 2015, 11:03 a.m.,

Review Request 33026: Revert "Make health check configurable"

2015-04-09 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33026/ --- Review request for Aurora, Brian Brazil and Kevin Sweeney. Bugs: AURORA-1266

Re: Review Request 33019: Added new (optional) paramter to ProcessBase for a statsd end-point, and submits a counter (increment) per "process.state" when statsd end-point provided

2015-04-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33019/#review79521 --- Master (6384710) is red with this patch. ./build-support/jenkins/b

Review Request 33019: Added new (optional) paramter to ProcessBase for a statsd end-point, and submits a counter (increment) per "process.state" when statsd end-point provided

2015-04-09 Thread Michael Schenck
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33019/ --- Review request for Aurora. Repository: aurora Description --- * Plotting