Re: Review Request 38072: Revert "Refer to shared task_configs table for job updates."

2015-09-02 Thread Aurora ReviewBot

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

Ship it!


Master (85f93a7) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot 
retry"

- Aurora ReviewBot


On Sept. 2, 2015, 10:07 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38072/
> ---
> 
> (Updated Sept. 2, 2015, 10:07 p.m.)
> 
> 
> Review request for Aurora and Maxim Khutornenko.
> 
> 
> Bugs: AURORA-1476
> https://issues.apache.org/jira/browse/AURORA-1476
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This patch switched the update store over to use part of the experimental 
> task store, which is affected by AURORA-1476.  Reverting until that issue is 
> fixed.
> 
> 
> Diffs
> -
> 
>   src/jmh/java/org/apache/aurora/benchmark/JobUpdates.java 
> 493f75dbf6b9403e5fb5bb48306336880d54 
>   src/main/java/org/apache/aurora/scheduler/storage/db/DbJobUpdateStore.java 
> 765213269c3014866ed8a5deafea9b6f667e1500 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/JobUpdateDetailsMapper.java
>  a3b04949f1726f110638e4f93ef45947cdb9e7f8 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/typehandlers/TaskConfigTypeHandler.java
>  PRE-CREATION 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/typehandlers/TypeHandlers.java
>  ed561c65947d41861d80229ad459392a4bceadf1 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/views/DbInstanceTaskConfig.java
>  f3fd7a96326408a990ab5014b86336759902f462 
>   src/main/java/org/apache/aurora/scheduler/storage/db/views/DbJobUpdate.java 
> 78703e92c932cd5e93ff0b70f2a0b6928f6b4003 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/views/DbJobUpdateDetails.java
>  3a5272471e15177a35397acea920198499d16f0b 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/views/DbJobUpdateInstructions.java
>  d19aa859bcc5da2c9c7d4ebbb8db307a1ea2883e 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/views/DbStoredJobUpdateDetails.java
>  8ec6d4710964fd64d24774a719baffa27502c607 
>   
> src/main/resources/org/apache/aurora/scheduler/storage/db/JobUpdateDetailsMapper.xml
>  69e11b69c4d8e0e2cba4c30ecc56077b2a3245ad 
>   src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql 
> 4df685a31b23ff40f42bb3a9b0278733cd722fd3 
>   
> src/test/java/org/apache/aurora/scheduler/storage/db/DbJobUpdateStoreTest.java
>  4c7133fa66bbe14002e2dc22cfaefa1585ab8a22 
> 
> Diff: https://reviews.apache.org/r/38072/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Bill Farner
> 
>



Re: Review Request 38072: Revert "Refer to shared task_configs table for job updates."

2015-09-02 Thread Maxim Khutornenko

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

Ship it!


Ship It!

- Maxim Khutornenko


On Sept. 2, 2015, 10:07 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38072/
> ---
> 
> (Updated Sept. 2, 2015, 10:07 p.m.)
> 
> 
> Review request for Aurora and Maxim Khutornenko.
> 
> 
> Bugs: AURORA-1476
> https://issues.apache.org/jira/browse/AURORA-1476
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This patch switched the update store over to use part of the experimental 
> task store, which is affected by AURORA-1476.  Reverting until that issue is 
> fixed.
> 
> 
> Diffs
> -
> 
>   src/jmh/java/org/apache/aurora/benchmark/JobUpdates.java 
> 493f75dbf6b9403e5fb5bb48306336880d54 
>   src/main/java/org/apache/aurora/scheduler/storage/db/DbJobUpdateStore.java 
> 765213269c3014866ed8a5deafea9b6f667e1500 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/JobUpdateDetailsMapper.java
>  a3b04949f1726f110638e4f93ef45947cdb9e7f8 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/typehandlers/TaskConfigTypeHandler.java
>  PRE-CREATION 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/typehandlers/TypeHandlers.java
>  ed561c65947d41861d80229ad459392a4bceadf1 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/views/DbInstanceTaskConfig.java
>  f3fd7a96326408a990ab5014b86336759902f462 
>   src/main/java/org/apache/aurora/scheduler/storage/db/views/DbJobUpdate.java 
> 78703e92c932cd5e93ff0b70f2a0b6928f6b4003 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/views/DbJobUpdateDetails.java
>  3a5272471e15177a35397acea920198499d16f0b 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/views/DbJobUpdateInstructions.java
>  d19aa859bcc5da2c9c7d4ebbb8db307a1ea2883e 
>   
> src/main/java/org/apache/aurora/scheduler/storage/db/views/DbStoredJobUpdateDetails.java
>  8ec6d4710964fd64d24774a719baffa27502c607 
>   
> src/main/resources/org/apache/aurora/scheduler/storage/db/JobUpdateDetailsMapper.xml
>  69e11b69c4d8e0e2cba4c30ecc56077b2a3245ad 
>   src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql 
> 4df685a31b23ff40f42bb3a9b0278733cd722fd3 
>   
> src/test/java/org/apache/aurora/scheduler/storage/db/DbJobUpdateStoreTest.java
>  4c7133fa66bbe14002e2dc22cfaefa1585ab8a22 
> 
> Diff: https://reviews.apache.org/r/38072/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Bill Farner
> 
>