> On April 27, 2015, 11:05 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/mesos/DriverFactoryImpl.java, > > line 51 > > <https://reviews.apache.org/r/33243/diff/2/?file=943401#file943401line51> > > > > Remove `credentials.get()`, it will throw a NullPointerException in > > this branch.
Terrible, sorry for the oversight! - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33243/#review81745 ----------------------------------------------------------- On April 27, 2015, 10:44 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33243/ > ----------------------------------------------------------- > > (Updated April 27, 2015, 10:44 p.m.) > > > Review request for Aurora, Kevin Sweeney and Bill Farner. > > > Bugs: AURORA-1228 > https://issues.apache.org/jira/browse/AURORA-1228 > > > Repository: aurora > > > Description > ------- > > The scheduler now explicitly acknowledges updates. I left the structure as > is, per Bill's suggestion. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/mesos/DriverFactoryImpl.java > db7aa74c2dfca3d520d978d7837d1f2639e6c05c > src/main/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImpl.java > f40746b5712f21e551c06eeb1aa379ebdcdbc693 > src/test/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImplTest.java > 4ef23ea6ebd2c9587a8356507fcb49a36b9de219 > > Diff: https://reviews.apache.org/r/33243/diff/ > > > Testing > ------- > > Ran the unit tests and the end-to-end test. > > > Thanks, > > Ben Mahler > >