Re: Review Request 21943: Accept a job key for config loading.

2014-06-09 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21943/#review45119 --- Ship it! schweet - Joe Smith On May 29, 2014, 11:21 a.m., Antoin

Re: Review Request 21943: Accept a job key for config loading.

2014-06-09 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21943/#review45111 --- Ship it! ef yeah! - Brian Wickman On May 29, 2014, 6:21 p.m., An

Re: Review Request 21943: Accept a job key for config loading.

2014-05-29 Thread Antoine Tollenaere
> On May 29, 2014, 12:49 a.m., Mark Chu-Carroll wrote: > > Thank you for this! Long overdue; I've been meaning to get to this, but > > never had time. Yes I was embarrassed with these old todos for myself. I can't merge the change though, you'll have to do it for me. - Antoine

Re: Review Request 21943: Accept a job key for config loading.

2014-05-29 Thread Antoine Tollenaere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21943/ --- (Updated May 29, 2014, 6:21 p.m.) Review request for Aurora, Mark Chu-Carroll a

Re: Review Request 21943: Accept a job key for config loading.

2014-05-28 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21943/#review44210 --- Ship it! Thank you for this! Long overdue; I've been meaning to get

Review Request 21943: Accept a job key for config loading.

2014-05-27 Thread Antoine Tollenaere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21943/ --- Review request for Aurora, Mark Chu-Carroll and Brian Wickman. Repository: auro