----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9860/#review17975 -----------------------------------------------------------
There was lots of back and forth about this topic. Is this patch still required? - Chip Childers On March 15, 2013, 1:59 p.m., Kelven Yang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9860/ > ----------------------------------------------------------- > > (Updated March 15, 2013, 1:59 p.m.) > > > Review request for cloudstack and Chip Childers. > > > Description > ------- > > log4j in KVM is not initialized through main(), instead, the agent instance > is initialized through invokation through java reflection. Initialize log4j > in a proper place > > > Diffs > ----- > > > Diff: https://reviews.apache.org/r/9860/diff/ > > > Testing > ------- > > Manually tested in my local KVM environment > > > Thanks, > > Kelven Yang > >
