----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31468/#review74223 -----------------------------------------------------------
Ship it! Ship It! - Jaideep dhok On Feb. 26, 2015, 5:45 a.m., Amareshwari Sriramadasu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31468/ > ----------------------------------------------------------- > > (Updated Feb. 26, 2015, 5:45 a.m.) > > > Review request for lens, Jaideep dhok and Rajat Khandelwal. > > > Bugs: LENS-348 > https://issues.apache.org/jira/browse/LENS-348 > > > Repository: lens > > > Description > ------- > > Removes synchronized on getMetrics() and getStatisticsService(), which are > not required. > > The only class level locks now are : > public synchronized void init(HiveConf hiveConf) > public synchronized void stop() > public synchronized void start() > > > Diffs > ----- > > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java > 82cfaf4 > > Diff: https://reviews.apache.org/r/31468/diff/ > > > Testing > ------- > > mvn clean package -DskipTests works. Running all tests, will update once done. > > > Thanks, > > Amareshwari Sriramadasu > >
