Re: Review Request 16767: Begin cleanup of StateManager interface by removing state change via Query.

2014-01-14 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16767/ --- (Updated Jan. 14, 2014, 9:51 p.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 16767: Begin cleanup of StateManager interface by removing state change via Query.

2014-01-14 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16767/#review31779 --- Ship it! Ship It! - Kevin Sweeney On Jan. 14, 2014, 11:30 a.m.,

Re: Review Request 16767: Begin cleanup of StateManager interface by removing state change via Query.

2014-01-14 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16767/ --- (Updated Jan. 14, 2014, 7:30 p.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 16767: Begin cleanup of StateManager interface by removing state change via Query.

2014-01-13 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16767/ --- (Updated Jan. 13, 2014, 11:35 p.m.) Review request for Aurora. Bugs: AURORA-2

Re: Review Request 16767: Begin cleanup of StateManager interface by removing state change via Query.

2014-01-13 Thread Bill Farner
> On Jan. 10, 2014, 9:01 p.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/state/StateManager.java, line 40 > > > > > > A bit of explanation of what "cas" stand for would not hurt here as > >

Re: Review Request 16767: Begin cleanup of StateManager interface by removing state change via Query.

2014-01-10 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16767/#review31538 --- Ship it! src/main/java/org/apache/aurora/scheduler/state/StateMana