----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31477/#review74239 -----------------------------------------------------------
Can you add a unit test doing acquire and release on a closed session? - Amareshwari Sriramadasu On Feb. 26, 2015, 8:13 a.m., Jaideep dhok wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31477/ > ----------------------------------------------------------- > > (Updated Feb. 26, 2015, 8:13 a.m.) > > > Review request for lens and Amareshwari Sriramadasu. > > > Bugs: LENS-300 > https://issues.apache.org/jira/browse/LENS-300 > > > Repository: lens > > > Description > ------- > > Added null checks in acquire(string), release(string) and getSession > > > Diffs > ----- > > lens-server/src/main/java/org/apache/lens/server/LensService.java > 14b7b19afb15195cc874499d5e895c9067d848fa > > Diff: https://reviews.apache.org/r/31477/diff/ > > > Testing > ------- > > > Thanks, > > Jaideep dhok > >
