----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14467/#review27181 -----------------------------------------------------------
Changes have been committed to 4.2. But the patch doesn't clean apply on master, can you please update it accordingly. - Likitha Shetty On Oct. 3, 2013, 11:31 a.m., Anshul Gangwar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14467/ > ----------------------------------------------------------- > > (Updated Oct. 3, 2013, 11:31 a.m.) > > > Review request for cloudstack and Devdeep Singh. > > > Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-4792 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-4792 > > > Repository: cloudstack-git > > > Description > ------- > > added the connection and socket timeout parameters for SMTP and sending > message in new thread so that HA doesn't get blocked beacause of hang in > sending email alert > > > Diffs > ----- > > server/src/com/cloud/alert/AlertManagerImpl.java c437a8c > > Diff: https://reviews.apache.org/r/14467/diff/ > > > Testing > ------- > > tested manually on my local setup > > > Thanks, > > Anshul Gangwar > >
