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


Ship it!




Ship It!

- Maxim Khutornenko


On Jan. 26, 2016, 4:49 p.m., John Sirois wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42804/
> -----------------------------------------------------------
> 
> (Updated Jan. 26, 2016, 4:49 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen and Bill Farner.
> 
> 
> Bugs: AURORA-1254
>     https://issues.apache.org/jira/browse/AURORA-1254
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> This field in the thrift api `JobUpdateSettings` struct and its sibling
> in `UpdateConfig.restart_threshold` on the client side were deprecated
> in Aurora 0.11.0.
> 
>  NEWS                                                                         
>         | 3 +++
>  api/src/main/thrift/org/apache/aurora/gen/api.thrift                         
>         | 6 ------
>  docs/configuration-reference.md                                              
>         | 1 -
>  src/jmh/java/org/apache/aurora/benchmark/JobUpdates.java                     
>         | 1 -
>  
> src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java
>        | 6 ------
>  src/main/python/apache/aurora/client/api/updater_util.py                     
>         | 3 ---
>  src/main/python/apache/aurora/config/schema/base.py                          
>         | 1 -
>  
> src/main/resources/org/apache/aurora/scheduler/storage/db/JobUpdateDetailsMapper.xml
>  | 4 ----
>  src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql         
>         | 1 -
>  
> src/test/java/org/apache/aurora/scheduler/storage/db/DbJobUpdateStoreTest.java
>        | 1 -
>  
> src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
>    | 1 -
>  src/test/java/org/apache/aurora/scheduler/updater/JobUpdaterIT.java          
>         | 9 ++-------
>  src/test/java/org/apache/aurora/scheduler/updater/UpdateFactoryImplTest.java 
>         | 1 -
>  src/test/python/apache/aurora/client/api/test_api.py                         
>         | 1 -
>  src/test/python/apache/aurora/client/api/test_updater_util.py                
>         | 6 +++---
>  src/test/python/apache/aurora/client/cli/test_config_noun.py                 
>         | 1 -
>  src/test/python/apache/aurora/client/cli/util.py                             
>         | 2 --
>  17 files changed, 8 insertions(+), 40 deletions(-)
> 
> 
> Diffs
> -----
> 
>   NEWS f2798f6a2d6841606e99a93677067e58dc5cf5cc 
>   api/src/main/thrift/org/apache/aurora/gen/api.thrift 
> a93df2165c208c4963975aeb4f174df602baa476 
>   docs/configuration-reference.md e0eb908ea7f1b888a87d0179c91f14b3577ea3bd 
>   src/jmh/java/org/apache/aurora/benchmark/JobUpdates.java 
> 493f75dbf6b44449403e5fb5bb48306336880d54 
>   
> src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java
>  662cae179142b34896b11275bd3b425903f22e3e 
>   src/main/python/apache/aurora/client/api/updater_util.py 
> a3389f3dcea62683f459c2e5f8def2538b8cc336 
>   src/main/python/apache/aurora/config/schema/base.py 
> 7bc903c4434afb15d7005e1ac6bdea208e352a49 
>   
> src/main/resources/org/apache/aurora/scheduler/storage/db/JobUpdateDetailsMapper.xml
>  69e11b69c4d8e0e2cba4c30ecc56077b2a3245ad 
>   src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql 
> 803960fb551aa324a4737d45593d6eb420817525 
>   
> src/test/java/org/apache/aurora/scheduler/storage/db/DbJobUpdateStoreTest.java
>  d035513435b9b07027747bb13b7810061f489474 
>   
> src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
>  b9ce2ddba051ab2baed59108d70537d84a031934 
>   src/test/java/org/apache/aurora/scheduler/updater/JobUpdaterIT.java 
> 2024b2c50d5d1e44f3f95b915c8bcd58e39379cb 
>   
> src/test/java/org/apache/aurora/scheduler/updater/UpdateFactoryImplTest.java 
> c31e082ef3dd3a40d9cb7926aa277e86399a06bd 
>   src/test/python/apache/aurora/client/api/test_api.py 
> 59f34a58ce083d9e2bdf125a8baaf511ca6d3901 
>   src/test/python/apache/aurora/client/api/test_updater_util.py 
> 11b95c6730313e5d520486063e7979514cee34e1 
>   src/test/python/apache/aurora/client/cli/test_config_noun.py 
> d82a69d008d7aa291058a4a244308fdb13c1b6a5 
>   src/test/python/apache/aurora/client/cli/util.py 
> c7aa5cc5a5ea1da01fa75d4d22ff1256e27521b7 
> 
> Diff: https://reviews.apache.org/r/42804/diff/
> 
> 
> Testing
> -------
> 
> Locally green:
> ```
> ./build-support/jenkins/build.sh
> ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
> ```
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Reply via email to