----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24727/#review50723 -----------------------------------------------------------
src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java <https://reviews.apache.org/r/24727/#comment88585> Why this change? The lock is supposed to be already acquired by the time startJobUpdate is executed, hense the Lock instance in the argument set. How is it supposed to work now? - Maxim Khutornenko On Aug. 15, 2014, 3:19 a.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24727/ > ----------------------------------------------------------- > > (Updated Aug. 15, 2014, 3:19 a.m.) > > > Review request for Aurora and Maxim Khutornenko. > > > Bugs: AURORA-613 > https://issues.apache.org/jira/browse/AURORA-613 > > > Repository: aurora > > > Description > ------- > > Store a lock association with job updates. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/state/JobUpdater.java > f15344417f52fe5e909abfbba636c48277404fb4 > src/main/java/org/apache/aurora/scheduler/state/JobUpdaterImpl.java > 6bcdf620c993091999c6cccaeae023cb061cbd50 > src/main/java/org/apache/aurora/scheduler/storage/ForwardingStore.java > 3f083d60a882c6665d9891172edb9a5aeddade9b > src/main/java/org/apache/aurora/scheduler/storage/JobUpdateStore.java > c05833f30eaf79527599a7223791a6f4f5309388 > src/main/java/org/apache/aurora/scheduler/storage/db/DBJobUpdateStore.java > d659aa124bd702589952da1b19854a00862b0c86 > > src/main/java/org/apache/aurora/scheduler/storage/db/JobUpdateDetailsMapper.java > d590219e9ff873bc7b9b740759b024c463c523cf > src/main/java/org/apache/aurora/scheduler/storage/log/LogStorage.java > 342bab0426ebeeab0b3d2d038d98dba1de836231 > > src/main/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImpl.java > 3d291dd21892c20a8eb9388744c8b2f10a811554 > > src/main/java/org/apache/aurora/scheduler/storage/log/WriteAheadStorage.java > 2915ff00f3a2e3602414cedebbb0270e07dc869a > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java > 0802ee08601f5b9747f99823679d92af59a76cbc > > src/main/resources/org/apache/aurora/scheduler/storage/db/JobUpdateDetailsMapper.xml > 17c58b1a07f2fcef7fe8502540b347542b603e60 > src/main/resources/org/apache/aurora/scheduler/storage/db/schema.sql > 1cf803fe019290c042e5a73824e39a12072b2431 > src/main/thrift/org/apache/aurora/gen/storage.thrift > 9f8378ea3f5d386c3177207296bf4b436b730b45 > src/main/thrift/org/apache/aurora/gen/storage_local.thrift > becfd7528610a32af907489d021319d9b371c332 > src/test/java/org/apache/aurora/scheduler/state/JobUpdaterImplTest.java > 1f985fb75cff7e120bce9e60b91a19c7e19b9c2a > src/test/java/org/apache/aurora/scheduler/storage/backup/RecoveryTest.java > 2f2c3e12657e9af1edf819ff95d0da5db0c5de4b > > src/test/java/org/apache/aurora/scheduler/storage/db/DBJobUpdateStoreTest.java > f695b85514bcc5cedb16e962124af3db052cb17a > src/test/java/org/apache/aurora/scheduler/storage/db/DbLockStoreTest.java > ae4cef42d67556a22ea45eaa6d7542915924ffd1 > src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java > ebcb9103d75909080f5b6a69db3a1bf46cfd9780 > > src/test/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImplTest.java > bee9c9c1fb43c5703c291edc51cb1bb73aefc8e5 > > src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java > 7dbf97a6cecef928f563e76d37488816ca91872a > > Diff: https://reviews.apache.org/r/24727/diff/ > > > Testing > ------- > > ./gradlew build -Pq > > > Thanks, > > Bill Farner > >