Review Request 40391: Introduce a utility class to read executor configurations in JSON.

2015-11-17 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40391/ --- Review request for Aurora, John Sirois and Zameer Manji. Bugs: AURORA-1376

Re: Review Request 40323: Improve the isolation in the thrift build.

2015-11-17 Thread John Sirois
> On Nov. 16, 2015, 3:18 p.m., Zameer Manji wrote: > > build-support/thrift/Makefile, line 28 > > > > > > The ticket says this patch was committed in 0.9.2. Would it be more > > productive to upgrade to 0.9.2+

Re: Review Request 40323: Improve the isolation in the thrift build.

2015-11-17 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40323/#review106868 --- Ship it! Master (86761e8) is green with this patch.

Re: Review Request 40323: Improve the isolation in the thrift build.

2015-11-17 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40323/ --- (Updated Nov. 17, 2015, 7:56 a.m.) Review request for Aurora, Bill Farner and

Re: Review Request 40391: Introduce a utility class to read executor configurations in JSON.

2015-11-17 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40391/#review106878 --- Ship it! Master (7b1e848) is green with this patch.

Re: Review Request 40391: Introduce a utility class to read executor configurations in JSON.

2015-11-17 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40391/#review106901 --- Ship it! Master (7b1e848) is green with this patch.

Re: Review Request 40391: Introduce a utility class to read executor configurations in JSON.

2015-11-17 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40391/#review106895 --- Ship it! Ship It! - John Sirois On Nov. 17, 2015, 10:51 a.m.,

Re: Review Request 40391: Introduce a utility class to read executor configurations in JSON.

2015-11-17 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40391/ --- (Updated Nov. 17, 2015, 9:51 a.m.) Review request for Aurora, John Sirois and

Re: Review Request 40391: Introduce a utility class to read executor configurations in JSON.

2015-11-17 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40391/#review106946 --- Ship it! Ship It! - Zameer Manji On Nov. 17, 2015, 9:51 a.m.,