Re: Review Request 51973: Fix for AURORA-1739, enables golang thrift bindings to create jobs

2016-09-16 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51973/#review149273 --- Ship it! Ship It! - Joshua Cohen On Sept. 16, 2016, 8:49

Re: Review Request 51973: Fix for AURORA-1739, enables golang thrift bindings to create jobs

2016-09-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51973/#review149272 --- Ship it! Master (783baae) is green with this patch.

Re: Review Request 51973: Fix for AURORA-1739, enables golang thrift bindings to create jobs

2016-09-16 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51973/#review149271 --- Ship it! Ship It! - Maxim Khutornenko On Sept. 16, 2016,

Re: Review Request 51973: Fix for AURORA-1739, enables golang thrift bindings to create jobs

2016-09-16 Thread Renan DelValle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51973/ --- (Updated Sept. 16, 2016, 1:49 p.m.) Review request for Aurora, Joshua Cohen

Re: Review Request 51973: Fix for AURORA-1739, enables golang thrift bindings to create jobs

2016-09-16 Thread Renan DelValle
> On Sept. 16, 2016, 1:42 p.m., Maxim Khutornenko wrote: > > Not that it changes much but since it's technically a schema change a > > release note would be great. Doh! Can't believe I forgot. Coming right up. - Renan --- This is an

Re: Review Request 51973: Fix for AURORA-1739, enables golang thrift bindings to create jobs

2016-09-16 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51973/#review149266 --- Not that it changes much but since it's technically a schema