Re: Review Request: Fix for deadlock in UserManager

2012-11-02 Thread Yuri Zelikov
> On Nov. 2, 2012, 3:37 p.m., Yuri Zelikov wrote: > > Committed the fix as suggested by Andrew. > > Committ r1405010 @Vicenet Please close the code review - Yuri --- This is an automatically generated e-mail. To reply, visit: https://r

Re: Review Request: Fix for deadlock in UserManager

2012-11-02 Thread Yuri Zelikov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7253/#review13039 --- Ship it! Committed the fix as suggested by Andrew. Committ r1405010

Re: Review Request: Fix for deadlock in UserManager

2012-10-21 Thread Vicente J. Ruiz Jurado
> On Oct. 1, 2012, 11:25 a.m., Yuri Zelikov wrote: > > I asked Andrew if he experienced similar issues on wiab.pro and he replied > > that yes and his solution was to remove synchronization on userManager in > > ClientFrontendImpl. > > I also tend to think that this is the probable source of de

Re: Review Request: Fix for deadlock in UserManager

2012-10-21 Thread Yuri Zelikov
> On Oct. 1, 2012, 11:25 a.m., Yuri Zelikov wrote: > > I asked Andrew if he experienced similar issues on wiab.pro and he replied > > that yes and his solution was to remove synchronization on userManager in > > ClientFrontendImpl. > > I also tend to think that this is the probable source of de

Re: Review Request: Fix for deadlock in UserManager

2012-10-02 Thread Vicente J. Ruiz Jurado
> On Oct. 1, 2012, 11:25 a.m., Yuri Zelikov wrote: > > I asked Andrew if he experienced similar issues on wiab.pro and he replied > > that yes and his solution was to remove synchronization on userManager in > > ClientFrontendImpl. > > I also tend to think that this is the probable source of de

Re: Review Request: Fix for deadlock in UserManager

2012-10-01 Thread Yuri Zelikov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7253/#review12062 --- I asked Andrew if he experienced similar issues on wiab.pro and he re

Review Request: Fix for deadlock in UserManager

2012-09-25 Thread Vicente J. Ruiz Jurado
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7253/ --- Review request for wave, Michael MacFadden, Yuri Zelikov, and Ali Lown. Descript