Has this actually been applied yet?
On Fri, Jul 26, 2013 at 10:33 AM, John Burwell <jburw...@basho.com> wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12963/#review23950 > ----------------------------------------------------------- > > Ship it! > > > Looks clean to me. Good cleanups. > > - John Burwell > > > On July 26, 2013, 9:07 a.m., Thomas O'Dowd wrote: > > > > ----------------------------------------------------------- > > This is an automatically generated e-mail. To reply, visit: > > https://reviews.apache.org/r/12963/ > > ----------------------------------------------------------- > > > > (Updated July 26, 2013, 9:07 a.m.) > > > > > > Review request for cloudstack. > > > > > > Repository: cloudstack-git > > > > > > Description > > ------- > > > > Simple text only fixes to update some typos I noticed in log messages > when testing 4.2. > > > > > > Diffs > > ----- > > > > server/src/com/cloud/configuration/ConfigurationManagerImpl.java > 6c32c55 > > > > Diff: https://reviews.apache.org/r/12963/diff/ > > > > > > Testing > > ------- > > > > Although this patch is against master, I cherry picked it locally to > test against 4.2 and did a clean compile. I was able to see the updated > message for "in stead" but I didn't check the other messages. > > > > > > Thanks, > > > > Thomas O'Dowd > > > > > >
