> On Nov. 20, 2015, 6 p.m., Amareshwari Sriramadasu wrote: > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java, > > line 269 > > <https://reviews.apache.org/r/40537/diff/1/?file=1134230#file1134230line269> > > > > Why is private removed? > > Rajat Khandelwal wrote: > Acquiring lock in test case to get snapshot of running, queued, waiting > queries atomically. > > Amareshwari Sriramadasu wrote: > We should not open up locks even for tests. We should add getters if > required.
Changed to add a method in query service. - Rajat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40537/#review107353 ----------------------------------------------------------- On Nov. 20, 2015, 6:22 p.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40537/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2015, 6:22 p.m.) > > > Review request for lens. > > > Bugs: LENS-869 > https://issues.apache.org/jira/browse/LENS-869 > > > Repository: lens > > > Description > ------- > > > Diffs > ----- > > lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java > c96ef200f710076e246f8ac2095f398a5b0da985 > lens-server/src/main/java/org/apache/lens/server/LensServerConf.java > 6db720d0f62425454f6a9e18a0365897c08d1980 > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java > fdc8bfd315e29a3f74232767dc4776f155bb1d22 > > lens-server/src/main/java/org/apache/lens/server/query/collect/DefaultEstimatedQueryCollection.java > 908b86fa3a2f26ee429cf32ccd6b0dd4b52beb8b > lens-server/src/test/java/org/apache/lens/server/LensJerseyTest.java > 3dad050e255cbb177a46b6a14f49acf33e9a51bd > > lens-server/src/test/java/org/apache/lens/server/query/TestQueryConstraints.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/40537/diff/ > > > Testing > ------- > > > Thanks, > > Rajat Khandelwal > >
