Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-05 Thread Rajat Khandelwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58986/ --- (Updated May 5, 2017, 5 p.m.) Review request for lens. Bugs: LENS-1418

Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-05 Thread Puneet Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58986/#review174017 --- Ship it! Ship It! - Puneet Gupta On May 5, 2017, 11:17

Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-05 Thread Rajat Khandelwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58986/ --- (Updated May 5, 2017, 4:47 p.m.) Review request for lens. Bugs: LENS-1418

Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-05 Thread Rajat Khandelwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58986/ --- (Updated May 5, 2017, 2:35 p.m.) Review request for lens. Bugs: LENS-1418

Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-05 Thread Puneet Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58986/#review174011 ---

Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-05 Thread Rajat Khandelwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58986/ --- (Updated May 5, 2017, 1:52 p.m.) Review request for lens. Bugs: LENS-1418

Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-05 Thread Rajat Khandelwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58986/ --- (Updated May 5, 2017, 12:23 p.m.) Review request for lens. Bugs: LENS-1418

Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-05 Thread Rajat Khandelwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58986/ --- (Updated May 5, 2017, 12:22 p.m.) Review request for lens. Bugs: LENS-1418

Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-05 Thread Rajat Khandelwal
> On May 5, 2017, 10:17 a.m., Puneet Gupta wrote: > > lens-client/src/main/java/org/apache/lens/client/LensClient.java > > Line 161 (original), 169 (patched) > > > > > > Are we clearing the statementMap at any point

Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-05 Thread Rajat Khandelwal
> On May 5, 2017, 10:17 a.m., Puneet Gupta wrote: > > lens-client/src/main/java/org/apache/lens/client/LensClient.java > > Line 161 (original), 169 (patched) > > > > > > Are we clearing the statementMap at any point

Re: Review Request 58986: LENS-1418: LensClient is not thread safe

2017-05-04 Thread Puneet Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58986/#review173999 --- lens-client/src/main/java/org/apache/lens/client/LensClient.java