Re: Review Request 29117: Adding thrift API changes document.

> On Feb. 12, 2015, 4:41 p.m., David McLaughlin wrote: > > docs/thrift-deprecation.md, line 20 > > > > > > Sorry I missed this first review, but AFAIK this isn't true for dynamic > > languages (incl. our own scheduler

Re: Review Request 29117: Adding thrift API changes document.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29117/#review72295 --- Ship it! Master (b62ec61) is green with this patch. ./build-suppo

Re: Review Request 29117: Adding thrift API changes document.

> On Feb. 13, 2015, 12:41 a.m., David McLaughlin wrote: > > docs/thrift-deprecation.md, line 20 > > > > > > Sorry I missed this first review, but AFAIK this isn't true for dynamic > > languages (incl. our own scheduler

Re: Review Request 29117: Adding thrift API changes document.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29117/#review72291 --- docs/thrift-deprecation.md

Re: Review Request 29117: Adding thrift API changes document.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29117/ --- (Updated Feb. 13, 2015, 12:35 a.m.) Review request for Aurora and Bill Farner.

Re: Review Request 29117: Adding thrift API changes document.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29117/#review70652 --- Ship it! Ship It! - Bill Farner On Jan. 21, 2015, 10:59 p.m., Ma

Re: Review Request 29117: Adding thrift API changes document.

> On Jan. 9, 2015, 6:44 p.m., Bill Farner wrote: > > docs/thrift-deprecation.md, line 31 > > > > > > There should be an item about logging and signaling in API responses > > when deprecated fields are used. > > Maxim

Re: Review Request 29117: Adding thrift API changes document.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29117/#review69071 --- This patch does not apply cleanly on master (116ee2d), do you need t

Re: Review Request 29117: Adding thrift API changes document.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29117/ --- (Updated Jan. 21, 2015, 10:59 p.m.) Review request for Aurora, Kevin Sweeney an

Re: Review Request 29117: Adding thrift API changes document.

> On Jan. 9, 2015, 6:44 p.m., Bill Farner wrote: > > docs/developing-aurora-client.md, line 124 > > > > > > s/thrift/Thrift/ Uppercased everywhere. > On Jan. 9, 2015, 6:44 p.m., Bill Farner wrote: > > docs/thrift-de

Re: Review Request 29117: Adding thrift API changes document.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29117/#review67471 --- docs/developing-aurora-client.md

Re: Review Request 29117: Adding thrift API changes document.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29117/ --- (Updated Jan. 6, 2015, 11:30 p.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 29117: Adding thrift API changes document.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29117/#review65255 --- Ship it! Master (95ad0fa) is green with this patch. ./build-suppo

Re: Review Request 29117: Adding thrift API changes document.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29117/ --- (Updated Dec. 16, 2014, 10:29 p.m.) Review request for Aurora and Kevin Sweeney