You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@aurora.apache.org by Bill Farner <wf...@apache.org> on 2014/10/01 00:08:22 UTC

Re: Review Request 26158: Perform job update no-op detection in SchedulerThriftInterface.

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

(Updated Sept. 30, 2014, 10:08 p.m.)


Review request for Aurora and Maxim Khutornenko.


Changes
-------

Rebase after conflict on master.


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


Repository: aurora


Description
-------

Perform job update no-op detection in SchedulerThriftInterface.


Diffs (updated)
-----

  src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 2847bdbb773f4d5a7f31b52ceb806880e736a259 
  src/main/java/org/apache/aurora/scheduler/updater/JobDiff.java 5daefff05edeaed236b76419ab65cbc3cddd8bac 
  src/main/java/org/apache/aurora/scheduler/updater/JobUpdateController.java f37fbef4c50cfc2d98bc1fd89376a210f4fa8643 
  src/main/java/org/apache/aurora/scheduler/updater/JobUpdateControllerImpl.java 97bc1dfbbd83d30d6e94b70902f225f2c92acb77 
  src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java 12b19fd333a95d0d3d1864bac629d011c271ee52 
  src/test/java/org/apache/aurora/scheduler/updater/JobUpdaterIT.java 7254588b55df9a12217c8ec172abc5976892497e 

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


Testing
-------

./gradlew build -Pq

Ran a no-op update in vagrant and saw expected behavior.


Thanks,

Bill Farner