Re: Review Request 36305: Remove restriction that -hostname must resolve.

2015-07-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36305/#review90888 --- Master (62a432a) is green with this patch.

Review Request 36305: Remove restriction that -hostname must resolve.

2015-07-08 Thread Brian Brazil
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36305/ --- Review request for Aurora, Stephan Erb and Bill Farner. Repository: aurora

Re: Review Request 36320: Enable debugging tools on the scheduler in vagrant.

2015-07-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36320/#review90951 --- Ship it! Master (62a432a) is green with this patch.

Re: Review Request 36064: Perform leader redirect using HTTP status code 307

2015-07-08 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36064/#review90931 --- Friendly ping. Anything still missing here? - Stephan Erb On

Review Request 36319: DB task store: Remove invalid constraint on task_config_metadata.

2015-07-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36319/ --- Review request for Aurora and Zameer Manji. Bugs: AURORA-1383

Re: Review Request 36320: Enable debugging tools on the scheduler in vagrant.

2015-07-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36320/ --- (Updated July 8, 2015, 5:57 p.m.) Review request for Aurora and Zameer Manji.

Re: Review Request 36319: DB task store: Remove invalid constraint on task_config_metadata.

2015-07-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36319/#review90944 --- Ship it! Master (62a432a) is green with this patch.

Re: Review Request 36064: Perform leader redirect using HTTP status code 307

2015-07-08 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36064/#review90948 --- Ship it! Apologies, I was moving across the country last week and

Re: Review Request 36319: DB task store: Remove invalid constraint on task_config_metadata.

2015-07-08 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36319/#review90932 --- Ship it! Ship It! - Zameer Manji On July 8, 2015, 10:46 a.m.,

Review Request 36320: Enable debugging tools on the scheduler in vagrant.

2015-07-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36320/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description

Re: Review Request 36320: Enable debugging tools on the scheduler in vagrant.

2015-07-08 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36320/#review90938 --- Ship it! Ship It! - Zameer Manji On July 8, 2015, 10:57 a.m.,

Review Request 36329: Add a stat that tracks uncaught exceptions in pubsub event handlers.

2015-07-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36329/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description

Re: Review Request 36305: Remove restriction that -hostname must resolve.

2015-07-08 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36305/#review90989 --- Ship it! Ship It! - Stephan Erb On July 8, 2015, 2:53 p.m.,

Re: Review Request 36329: Add a stat that tracks uncaught exceptions in pubsub event handlers.

2015-07-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36329/ --- (Updated July 8, 2015, 8:57 p.m.) Review request for Aurora and Zameer Manji.

Re: Review Request 36329: Add a stat that tracks uncaught exceptions in pubsub event handlers.

2015-07-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36329/#review90995 --- Ship it! Master (144cf60) is green with this patch.

Re: Review Request 36305: Remove restriction that -hostname must resolve.

2015-07-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36305/#review91000 --- Ship it!

Re: Review Request 34337: Add Docker Parameters

2015-07-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34337/#review90994 --- Only nits remaining, and one request for test coverage. One final

Re: Review Request 36064: Perform leader redirect using HTTP status code 307

2015-07-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36064/#review91003 --- Ship it! Ship It! - Bill Farner On July 5, 2015, 7:38 p.m.,

Re: Review Request 36329: Add a stat that tracks uncaught exceptions in pubsub event handlers.

2015-07-08 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36329/#review91039 --- Ship it! Ship It! - Zameer Manji On July 8, 2015, 1:57 p.m.,

Re: Review Request 36338: Don't reject update requests scoped to already-updated instances.

2015-07-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36338/#review91025 --- Ship it! Master (8efcd06) is green with this patch.

Review Request 36338: Don't reject update requests scoped to already-updated instances.

2015-07-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36338/ --- Review request for Aurora and Zameer Manji. Bugs: AURORA-1332