Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-25 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/ --- (Updated March 25, 2014, 11:08 a.m.) Review request for Aurora, Suman Karumuri

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-25 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/ --- (Updated March 25, 2014, 11:06 a.m.) Review request for Aurora, Suman Karumuri

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-25 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/ --- (Updated March 25, 2014, 11:03 a.m.) Review request for Aurora, Suman Karumuri

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-25 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/#review38476 --- Ship it! Can you drop a comment in the code about why we don't appl

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-25 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/ --- (Updated March 25, 2014, 10:28 a.m.) Review request for Aurora, Suman Karumuri

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-21 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/#review38135 --- Ship it! Ship It! - Suman Karumuri On March 20, 2014, 6:29 p.m.,

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/#review37974 --- Ship it! Just to be extra sure, can you do a manual smoke test of t

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-20 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/ --- (Updated March 20, 2014, 11:29 a.m.) Review request for Aurora, Suman Karumuri

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-20 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/ --- (Updated March 20, 2014, 11:26 a.m.) Review request for Aurora, Suman Karumuri

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-20 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/ --- (Updated March 20, 2014, 11:22 a.m.) Review request for Aurora, Suman Karumuri

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/#review37924 --- Update or discard? - Bill Farner On March 1, 2014, 1:27 a.m., Kev

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-06 Thread Suman Karumuri
> On March 1, 2014, 5:23 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/thrift/ThriftModule.java, line 40 > > > > > > I can't think of a good reason to not apply this to /, can you? If so, > >

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-03 Thread Bill Farner
> On March 1, 2014, 5:23 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/thrift/ThriftModule.java, line 40 > > > > > > I can't think of a good reason to not apply this to /, can you? If so, > >

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-03 Thread Kevin Sweeney
> On March 1, 2014, 9:23 a.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/thrift/ThriftModule.java, line 40 > > > > > > I can't think of a good reason to not apply this to /, can you? If so, > >

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-01 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/#review35907 --- src/main/java/org/apache/aurora/scheduler/thrift/ThriftModule.java

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-02-28 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/ --- (Updated Feb. 28, 2014, 5:27 p.m.) Review request for Aurora, Suman Karumuri an

Review Request 18658: GZIP HTTP thrift API responses

2014-02-28 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18658/ --- Review request for Aurora, Suman Karumuri and Bill Farner. Repository: aurora