Re: Review Request 17723: Implementing support for DRAINING state.

2014-02-13 Thread Maxim Khutornenko

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17723/
---

(Updated Feb. 13, 2014, 3:49 p.m.)


Review request for Aurora, Kevin Sweeney and Bill Farner.


Changes
---

Rebased over master.


Bugs: AURORA-82
https://issues.apache.org/jira/browse/AURORA-82


Repository: aurora


Description
---

Implementing support for DRAINING state.


Diffs (updated)
-

  src/main/java/org/apache/aurora/scheduler/async/RescheduleCalculator.java 
b7e644782ff2e97b8cf70421751f97c7b4334792 
  src/main/java/org/apache/aurora/scheduler/async/TaskTimeout.java 
609e8ccb08d13f562ed6fa1b0abc71c3a054ae7a 
  src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java 
4cb5882140c51f1f1ce1bd51f03bbd11947f0b09 
  src/main/java/org/apache/aurora/scheduler/state/MaintenanceController.java 
0e2c17bd69adb47baac257f38a9263f02a4fddc3 
  src/main/java/org/apache/aurora/scheduler/state/TaskStateMachine.java 
0b39b841f369a2c01092d5eae5867a8b91166523 
  src/main/java/org/apache/aurora/scheduler/storage/StorageBackfill.java 
714bceb8b1cfabd8a4959d8b529be2880f4b53fd 
  src/main/thrift/org/apache/aurora/gen/api.thrift 
23883cf6a412c36b41b041df15b6883cd4155596 
  src/test/java/org/apache/aurora/scheduler/async/TaskTimeoutTest.java 
d4685f65f8695b20b8c52e2e983e57099f578bef 
  
src/test/java/org/apache/aurora/scheduler/state/MaintenanceControllerImplTest.java
 56b691e3787de4f30f3e613512ca3415337e9756 
  src/test/java/org/apache/aurora/scheduler/state/TaskStateMachineTest.java 
a2aaeeb8c49b050bf051e6574aa852116dd61578 
  src/test/java/org/apache/aurora/scheduler/storage/StorageBackfillTest.java 
c0cc0a472e61ceae22845541544c3e7d68ce7657 
  src/test/resources/org/apache/aurora/gen/api.thrift.md5 
2bdd038553107404c238a2fb8f28f9aa5bfc8549 

Diff: https://reviews.apache.org/r/17723/diff/


Testing
---

gradle clean test


Thanks,

Maxim Khutornenko



Re: Review Request 17723: Implementing support for DRAINING state.

2014-02-11 Thread Bill Farner

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17723/#review34226
---

Ship it!


Ship It!

- Bill Farner


On Feb. 4, 2014, 9:31 p.m., Maxim Khutornenko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17723/
> ---
> 
> (Updated Feb. 4, 2014, 9:31 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Bugs: AURORA-82
> https://issues.apache.org/jira/browse/AURORA-82
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Implementing support for DRAINING state.
> 
> 
> Diffs
> -
> 
>   src/main/java/org/apache/aurora/scheduler/async/RescheduleCalculator.java 
> a9ee32b6021b21d71db71b44e2eca2b984143718 
>   src/main/java/org/apache/aurora/scheduler/async/TaskTimeout.java 
> 64a1941a3b92e65c057d64edd9a823e4f89807d1 
>   src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java 
> 4cfe6fe14083d018a9143e0279d8f9a392a54019 
>   src/main/java/org/apache/aurora/scheduler/state/MaintenanceController.java 
> af10d44431b9bc0aec20a36a45ff74ccb1365736 
>   src/main/java/org/apache/aurora/scheduler/state/TaskStateMachine.java 
> ebccc7479b0f632e585010bf9943db8f75f5ca7c 
>   src/main/java/org/apache/aurora/scheduler/storage/StorageBackfill.java 
> aa54659f309eab41f4299fecb99dda719005c147 
>   src/main/thrift/org/apache/aurora/gen/api.thrift 
> 77cd09ec91afc98910a867c2632c790a8e0a72ce 
>   src/test/java/org/apache/aurora/scheduler/async/TaskTimeoutTest.java 
> d4685f65f8695b20b8c52e2e983e57099f578bef 
>   
> src/test/java/org/apache/aurora/scheduler/state/MaintenanceControllerImplTest.java
>  f7f7632315496af085ef2f3783e7bb87fc4d23be 
>   src/test/java/org/apache/aurora/scheduler/state/TaskStateMachineTest.java 
> cc7d59dca749f67686c41064eeda6e62a380ffeb 
>   src/test/java/org/apache/aurora/scheduler/storage/StorageBackfillTest.java 
> f6ece9c35d5610e5522f000f87002a2fd5614484 
>   src/test/resources/org/apache/aurora/gen/api.thrift.md5 
> 10fcf08f87abb8a0e3d2750237b482ebdd3ac65d 
> 
> Diff: https://reviews.apache.org/r/17723/diff/
> 
> 
> Testing
> ---
> 
> gradle clean test
> 
> 
> Thanks,
> 
> Maxim Khutornenko
> 
>



Re: Review Request 17723: Implementing support for DRAINING state.

2014-02-06 Thread Kevin Sweeney

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17723/#review33847
---

Ship it!


Ship It!

- Kevin Sweeney


On Feb. 4, 2014, 1:31 p.m., Maxim Khutornenko wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17723/
> ---
> 
> (Updated Feb. 4, 2014, 1:31 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Bugs: AURORA-82
> https://issues.apache.org/jira/browse/AURORA-82
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Implementing support for DRAINING state.
> 
> 
> Diffs
> -
> 
>   src/main/java/org/apache/aurora/scheduler/async/RescheduleCalculator.java 
> a9ee32b6021b21d71db71b44e2eca2b984143718 
>   src/main/java/org/apache/aurora/scheduler/async/TaskTimeout.java 
> 64a1941a3b92e65c057d64edd9a823e4f89807d1 
>   src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java 
> 4cfe6fe14083d018a9143e0279d8f9a392a54019 
>   src/main/java/org/apache/aurora/scheduler/state/MaintenanceController.java 
> af10d44431b9bc0aec20a36a45ff74ccb1365736 
>   src/main/java/org/apache/aurora/scheduler/state/TaskStateMachine.java 
> ebccc7479b0f632e585010bf9943db8f75f5ca7c 
>   src/main/java/org/apache/aurora/scheduler/storage/StorageBackfill.java 
> aa54659f309eab41f4299fecb99dda719005c147 
>   src/main/thrift/org/apache/aurora/gen/api.thrift 
> 77cd09ec91afc98910a867c2632c790a8e0a72ce 
>   src/test/java/org/apache/aurora/scheduler/async/TaskTimeoutTest.java 
> d4685f65f8695b20b8c52e2e983e57099f578bef 
>   
> src/test/java/org/apache/aurora/scheduler/state/MaintenanceControllerImplTest.java
>  f7f7632315496af085ef2f3783e7bb87fc4d23be 
>   src/test/java/org/apache/aurora/scheduler/state/TaskStateMachineTest.java 
> cc7d59dca749f67686c41064eeda6e62a380ffeb 
>   src/test/java/org/apache/aurora/scheduler/storage/StorageBackfillTest.java 
> f6ece9c35d5610e5522f000f87002a2fd5614484 
>   src/test/resources/org/apache/aurora/gen/api.thrift.md5 
> 10fcf08f87abb8a0e3d2750237b482ebdd3ac65d 
> 
> Diff: https://reviews.apache.org/r/17723/diff/
> 
> 
> Testing
> ---
> 
> gradle clean test
> 
> 
> Thanks,
> 
> Maxim Khutornenko
> 
>