aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Maxim Khutornenko" <ma...@apache.org>
Subject Re: Review Request 39143: Adding getJobUpdateDiff thrift API.
Date Fri, 09 Oct 2015 22:43:56 GMT

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

(Updated Oct. 9, 2015, 10:43 p.m.)


Review request for Aurora, David McLaughlin and Bill Farner.


Changes
-------

Making getJobUpdateDiff return task configs. Please, review.


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


Repository: aurora


Description (updated)
-------

Adding the API to return update instructions. Design doc: https://docs.google.com/document/d/1Fc_YhhV7fc4D9Xv6gJzpfooxbK4YWZcvzw6Bd3qVTL8


Diffs (updated)
-----

  api/src/main/thrift/org/apache/aurora/gen/api.thrift f56d7bdc5cdbb2fc84254c41328b01c1367c8343

  src/main/java/org/apache/aurora/scheduler/thrift/ReadOnlySchedulerImpl.java c09130bcd4deae80e475f531e5bc08dac475f0e8

  src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 4efcd2cd81290e919be5cf1de94c5aeb0851c67e

  src/main/java/org/apache/aurora/scheduler/updater/JobDiff.java a17337fb2df8e6e2a2905e88ff1f1eeb03e7d406

  src/test/java/org/apache/aurora/scheduler/thrift/Fixtures.java adcc02b3313220c728a9025b9aec787646f71058

  src/test/java/org/apache/aurora/scheduler/thrift/ReadOnlySchedulerImplTest.java 82e7330a83ff366fab81385608c62d92445f5e16

  src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java 83f65a73fc2053b138854be13a3b6c7748f80c2b

  src/test/java/org/apache/aurora/scheduler/thrift/aop/ForwardingThrift.java d25618c19615ee097c1f2c4b6fefc70c565c649c

  src/test/java/org/apache/aurora/scheduler/updater/JobDiffTest.java 64d45813d89d5f084aec70cd87aadb01de7f6d41


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


Testing
-------

./gradlew -Pq build


Thanks,

Maxim Khutornenko


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message