----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39576/#review105367 -----------------------------------------------------------
lens-server/src/main/java/org/apache/lens/server/BaseLensService.java (line 87) <https://reviews.apache.org/r/39576/#comment163927> Better to use AtomicIntger to hold the session count for a user. As of now incrementing the plain integer counter and decrementing it is not thread safe. lens-server/src/main/java/org/apache/lens/server/BaseLensService.java (lines 115 - 118) <https://reviews.apache.org/r/39576/#comment163930> Suggestion : Should we store this value (maxAllowedSessionsPerUser) in a member varaible rather than computing it each time when this method is called ? lens-server/src/main/java/org/apache/lens/server/BaseLensService.java (lines 120 - 122) <https://reviews.apache.org/r/39576/#comment163931> There might be another thread updating the session count for the same user at the same time. Should we do a strict check (atleast at user level) ? lens-server/src/main/java/org/apache/lens/server/BaseLensService.java (lines 187 - 194) <https://reviews.apache.org/r/39576/#comment163932> Not Thread safe. lens-server/src/main/java/org/apache/lens/server/BaseLensService.java (lines 278 - 284) <https://reviews.apache.org/r/39576/#comment163933> We need to handle the case where decrementSessionCountForUser, updateSessionsPerUser and isMaxSessionsLimitReachedPerUser when called for same user from different threads at the same time. - Puneet Gupta On Oct. 31, 2015, 12:24 a.m., Raju Bairishetti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39576/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2015, 12:24 a.m.) > > > Review request for lens, Amareshwari Sriramadasu and Rajat Khandelwal. > > > Bugs: LENS-833 > https://issues.apache.org/jira/browse/LENS-833 > > > Repository: lens > > > Description > ------- > > Failing the open session operation if user creates more sessions than > configured limit. > > > Diffs > ----- > > lens-api/src/main/resources/lens-errors.conf a582dc2 > > lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java > 7ee0749 > lens-server/src/main/java/org/apache/lens/server/BaseLensService.java > 0821fe7 > lens-server/src/main/resources/lensserver-default.xml 5f268cb > lens-server/src/test/java/org/apache/lens/server/TestServerMode.java > def5683 > > lens-server/src/test/java/org/apache/lens/server/auth/FooBarAuthenticationProvider.java > 8e22837 > > lens-server/src/test/java/org/apache/lens/server/metastore/TestMetastoreService.java > ca0fe56 > > lens-server/src/test/java/org/apache/lens/server/query/QueryAPIErrorResponseTest.java > 18a8c8d > lens-server/src/test/java/org/apache/lens/server/query/TestLensDAO.java > bc1463f > > lens-server/src/test/java/org/apache/lens/server/session/TestSessionResource.java > 3055ce5 > > lens-server/src/test/java/org/apache/lens/server/ui/TestSessionUIResource.java > 6f7c216 > src/site/apt/admin/config.apt 88c1489 > > Diff: https://reviews.apache.org/r/39576/diff/ > > > Testing > ------- > > > Thanks, > > Raju Bairishetti > >
