Re: Review Request 42756: [3/3] Convert thrift codegen to use ThriftGen.

2016-01-26 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42756/ --- (Updated Jan. 26, 2016, 9:01 p.m.) Review request for Aurora. Bugs: AURORA-11

Re: Review Request 42756: [3/3] Convert thrift codegen to use ThriftGen.

2016-01-26 Thread John Sirois
> On Jan. 25, 2016, 3:55 p.m., John Sirois wrote: > > NB: Please hold off on review pending an email to dev@ explaining this 3 RB > > proposal. Review away, accompanying proposal is now out for review: http://markmail.org/message/a6sdqcelgokw6mwz - John

Re: Review Request 42756: [3/3] Convert thrift codegen to use ThriftGen.

2016-01-26 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42756/ --- (Updated Jan. 26, 2016, 11:37 a.m.) Review request for Aurora. Bugs: AURORA-1

Re: Review Request 42756: [3/3] Convert thrift codegen to use ThriftGen.

2016-01-25 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42756/#review116219 --- Master (39815a3) is red with this patch. ./build-support/jenkins

Re: Review Request 42756: [3/3] Convert thrift codegen to use ThriftGen.

2016-01-25 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42756/ --- (Updated Jan. 25, 2016, 4:22 p.m.) Review request for Aurora. Changes ---

Re: Review Request 42756: [3/3] Convert thrift codegen to use ThriftGen.

2016-01-25 Thread John Sirois
> On Jan. 25, 2016, 3:55 p.m., Aurora ReviewBot wrote: > > Master (39815a3) is red with this patch. > > ./build-support/jenkins/build.sh > > > > :buildSrc:compileGroovy > > :buildSrc:processResources UP-TO-DATE > > :buildSrc:classes > > :buildSrc:jar > > :buildSrc:assemble > > :buildSrc:compil

Re: Review Request 42756: [3/3] Convert thrift codegen to use ThriftGen.

2016-01-25 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42756/#review116207 --- Master (39815a3) is red with this patch. ./build-support/jenkins

Re: Review Request 42756: [3/3] Convert thrift codegen to use ThriftGen.

2016-01-25 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42756/#review116204 --- NB: Please hold off on review pending an email to dev@ explaining