Re: Review Request 17666: Adding DRAINING state into api.thrift.

2014-02-03 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17666/#review33462 --- Ship it! Ship It! - Kevin Sweeney On Feb. 3, 2014, 10:16 a.m.,

Re: Review Request 17666: Adding DRAINING state into api.thrift.

2014-02-03 Thread Maxim Khutornenko
On Feb. 3, 2014, 7:22 p.m., Bill Farner wrote: You might want to do a bit more in this commit, specifically to ensure read sites handle the new state. You probably also want to rewrite the state in StorageBackfill. Also look out for another regression like

Re: Review Request 17666: Adding DRAINING state into api.thrift.

2014-02-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17666/ --- (Updated Feb. 3, 2014, 8:44 p.m.) Review request for Aurora, Kevin Sweeney and