----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62873/#review187597 -----------------------------------------------------------
Master (4a1fba3) is green with this patch. ./build-support/jenkins/build.sh However, it appears that it might lack test coverage. I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Oct. 10, 2017, 8:35 p.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62873/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2017, 8:35 p.m.) > > > Review request for Aurora and Jordan Ly. > > > Repository: aurora > > > Description > ------- > > This reduces the memory burden of loading a backup for recovery. Previously, > the backup file would be fully loaded into a `byte[]`, which may be very > large and fail to allocate. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/storage/backup/Recovery.java > b74de9b51fd01788a3970e56a534a3b9adcd8863 > > > Diff: https://reviews.apache.org/r/62873/diff/1/ > > > Testing > ------- > > > Thanks, > > Bill Farner > >
