----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7130/#review11907 -----------------------------------------------------------
Could you also please add warn log statement on the case when account can't be found? At the moment we just return NULL. - Alena Prokharchyk On Sept. 25, 2012, 7:56 a.m., Rohit Yadav wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7130/ > ----------------------------------------------------------- > > (Updated Sept. 25, 2012, 7:56 a.m.) > > > Review request for cloudstack, Abhinandan Prateek, Alena Prokharchyk, and > Alex Huang. > > > Description > ------- > > Issue: https://issues.apache.org/jira/browse/CLOUDSTACK-121 > > Download original patch and git am <patch>: > http://bhaisaab.org/patches/cloudstack/0001-CLOUDSTACK-121-Fix-NullPointerException-for-incorrec.patch > > > Diffs > ----- > > api/src/com/cloud/user/UserAccount.java 734e16b > server/src/com/cloud/user/AccountManagerImpl.java fa9fafb > > Diff: https://reviews.apache.org/r/7130/diff/ > > > Testing > ------- > > > Thanks, > > Rohit Yadav > >
