----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30178/#review69231 -----------------------------------------------------------
Ship it! src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java <https://reviews.apache.org/r/30178/#comment113859> This is only used in one place, inline? - Maxim Khutornenko On Jan. 22, 2015, 7:25 p.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30178/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2015, 7:25 p.m.) > > > Review request for Aurora, Kevin Sweeney and Steve Niemitz. > > > Bugs: AURORA-1045 > https://issues.apache.org/jira/browse/AURORA-1045 > > > Repository: aurora > > > Description > ------- > > The primary metric for success with this patch is to never interact with > `sessionValidator` within a `storage.write` closure. > > This had two outcomes: > 1. collapsing update-related RPC implementations for better DRY behavior > 2. refactoring `killTasks` > > (2) has a behavioral change, though i think it's the correct behavior. For > example, before this patch you could successfully kill all `PENDING` tasks, > as long as you happened to own those tasks. The new behavior denies these > requests for non-admin users regardless of the result of the query. > > > Diffs > ----- > > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java > ac92959f34a3b0962d6aa018dc82a5ac72ea1b34 > > src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java > ad9126c32893080e128d086ea3bfd7ad23d27b89 > > Diff: https://reviews.apache.org/r/30178/diff/ > > > Testing > ------- > > > Thanks, > > Bill Farner > >
