aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <>
Subject Re: Review Request 28026: Add more test coverage to SchedulerThriftInterface.
Date Wed, 19 Nov 2014 22:16:54 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Nov. 19, 2014, 10:16 p.m.)

Review request for Aurora, David McLaughlin and Zameer Manji.

Bugs: AURORA-937

Repository: aurora


This brings SchedulerThriftInterface to 99% instruction coverage and 96% branch coverage.
 The remaining pieces are legitimately difficult to capture.

I also uncovered a few minor bugs along the way, most noteworthy being AOP interceptor ordering
leading to the standard API response not always being applied.

Additionally, i removed a bunch of unnecessary exception handling that is now done by an AOP


  src/main/java/org/apache/aurora/auth/ eeebb78901a6c33e08ceb8e675c91f0b5f44bcbc

  src/main/java/org/apache/aurora/scheduler/storage/backup/ 4744dc9f202969906113ccb610bf17c94d188c43

  src/main/java/org/apache/aurora/scheduler/thrift/ b2b66acee9c0789f3660469d6d504b4510af5e79

  src/main/java/org/apache/aurora/scheduler/thrift/ 18e2bdfab6406761d7e7cbcec26d12fb28441fe0

  src/main/java/org/apache/aurora/scheduler/thrift/aop/ dca855c522d21924821fc47e636da39689aec4b7

  src/test/java/org/apache/aurora/scheduler/thrift/ 5c9ea6cf4eb4d99d94f5d61e784dd7c9c480798c




Bill Farner

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