----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36387/#review91541 -----------------------------------------------------------
Ship it! Is it not possible for an integration test to catch this issue? - Zameer Manji On July 9, 2015, 10:31 p.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36387/ > ----------------------------------------------------------- > > (Updated July 9, 2015, 10:31 p.m.) > > > Review request for Aurora and Zameer Manji. > > > Repository: aurora > > > Description > ------- > > This was broken in https://reviews.apache.org/r/36357/ > > I don't have a good answer for how to test this that isn't a carbon copy of > the module code. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/storage/db/DbModule.java > 49d8bcbccf3ae2df74099551fae42dbc5292b559 > > Diff: https://reviews.apache.org/r/36387/diff/ > > > Testing > ------- > > > Thanks, > > Bill Farner > >