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 16710: Support multiple simulataneous auth mechanisms.

2014-03-25 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16710/#review38467 --- Discarding this review for now. - Kevin Sweeney On Jan. 7, 2014,

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:08 a.m.) Review request for Aurora, Suman Karumuri

Re: Review Request 19431: Add instance specifier syntax to clientv2

2014-03-25 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19431/#review38480 --- Ping? - Mark Chu-Carroll On March 24, 2014, 2:58 p.m., Mark

Re: Review Request 19396: Implement client command logging with a unique token.

2014-03-25 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19396/#review38481 --- kevin, ping? - Mark Chu-Carroll On March 21, 2014, 3:06 p.m.,

Re: Review Request 19466: Add batching to the kill and killall commands.

2014-03-25 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19466/#review38484 --- Brian, ping? - Mark Chu-Carroll On March 24, 2014, 7:21 p.m.,

Re: Review Request 19466: Add batching to the kill and killall commands.

2014-03-25 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19466/#review38489 --- Brian, ping? - Mark Chu-Carroll On March 24, 2014, 7:21 p.m.,

Review Request 19627: Updates documentation to specify template, adds headers to pages that needed them, cleans up whitespace, and makes markdown for headers consistent.

2014-03-25 Thread Dave Lester
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19627/ --- Review request for Aurora. Repository: aurora Description --- Updates

Re: Review Request 19396: Implement client command logging with a unique token.

2014-03-25 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19396/#review38513 --- Ship it! Ship It! - Kevin Sweeney On March 21, 2014, 12:06

Re: Review Request 19627: Updates documentation to specify template, adds headers to pages that needed them, cleans up whitespace, and makes markdown for headers consistent.

2014-03-25 Thread Jake Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19627/#review38514 --- docs/README.md https://reviews.apache.org/r/19627/#comment70755

Re: Review Request 19396: Implement client command logging with a unique token.

2014-03-25 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19396/ --- (Updated March 25, 2014, 4:18 p.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 19627: Updates documentation to specify template, adds headers to pages that needed them, cleans up whitespace, and makes markdown for headers consistent.

2014-03-25 Thread Dave Lester
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19627/ --- (Updated March 25, 2014, 8:54 p.m.) Review request for Aurora and Mark

Re: Review Request 19627: Updates documentation to specify template, adds headers to pages that needed them, cleans up whitespace, and makes markdown for headers consistent.

2014-03-25 Thread Dave Lester
On March 25, 2014, 8:07 p.m., Mark Chu-Carroll wrote: docs/README.md, line 1 https://reviews.apache.org/r/19627/diff/1/?file=535599#file535599line1 Per jfarrell, These headers aren't needed: you can just put the following into config.rb: page /documentation/*;

Re: Review Request 19466: Add batching to the kill and killall commands.

2014-03-25 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19466/#review38543 --- Ship it! Ship It! - Brian Wickman On March 24, 2014, 11:21

Re: Review Request 19431: Add instance specifier syntax to clientv2

2014-03-25 Thread Brian Wickman
On March 24, 2014, 6:57 p.m., Mark Chu-Carroll wrote: src/main/python/apache/aurora/client/cli/options.py, lines 121-124 https://reviews.apache.org/r/19431/diff/4/?file=532893#file532893line121 all was never revealed to users, and we've decided to get rid of it. It will never

Re: Review Request 19431: Add instance specifier syntax to clientv2

2014-03-25 Thread Mark Chu-Carroll
Not uncaught: it gets handled exactly the way that all invalid parameters get handled - the exception gets thrown in the parser, it bubbles up, and gets caught by the framework and turned into an error: [sun-wukong incubator-aurora (instance-syntax)]$ ./dist/client.pex job kill

Re: Review Request 19550: Filtering out non-prod tasks for domain sla calculations.

2014-03-25 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19550/#review38555 --- Ping, Brian. - Maxim Khutornenko On March 21, 2014, 9:15 p.m.,

Re: Review Request 19466: Add batching to the kill and killall commands.

2014-03-25 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19466/ --- (Updated March 25, 2014, 7:24 p.m.) Review request for Aurora, Maxim

Review Request 19648: Preserving sandbox deleted task history.

2014-03-25 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19648/ --- Review request for Aurora, Kevin Sweeney, Suman Karumuri, and Bill Farner.