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 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.,