----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62604/#review186360 -----------------------------------------------------------
Master (79ff364) 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 Sept. 27, 2017, 12:25 a.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62604/ > ----------------------------------------------------------- > > (Updated Sept. 27, 2017, 12:25 a.m.) > > > Review request for Aurora and Jordan Ly. > > > Repository: aurora > > > Description > ------- > > Came across this while looking at storage access patterns. Not a > particularly hot code path, but this is more efficient and concise. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/state/StateManagerImpl.java > 6b780ec90fed846531162eed5e86336e598e33a3 > > > Diff: https://reviews.apache.org/r/62604/diff/2/ > > > Testing > ------- > > > Thanks, > > Bill Farner > >
