----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36614/#review92377 -----------------------------------------------------------
lens-server/src/main/java/org/apache/lens/server/LogResource.java (line 76) <https://reviews.apache.org/r/36614/#comment146508> Agree, but we can close input stream. Can we check StreamingOutput() in QueryExecutionServiceExecImpl class? - Raju Bairishetti On July 20, 2015, 11:17 a.m., Amareshwari Sriramadasu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36614/ > ----------------------------------------------------------- > > (Updated July 20, 2015, 11:17 a.m.) > > > Review request for lens and Himanshu Gahlaut. > > > Bugs: LENS-670 > https://issues.apache.org/jira/browse/LENS-670 > > > Repository: lens > > > Description > ------- > > Adds log resource which is added through configuration > > > Diffs > ----- > > lens-server/src/main/java/org/apache/lens/server/LogResource.java > PRE-CREATION > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java > 47d27d0 > lens-server/src/main/java/org/apache/lens/server/util/UtilityMethods.java > 715347f > lens-server/src/main/resources/lensserver-default.xml 2b540b8 > lens-server/src/test/java/org/apache/lens/server/LensJerseyTest.java > fb3e67b > lens-server/src/test/java/org/apache/lens/server/TestLensApplication.java > a267fe0 > lens-server/src/test/java/org/apache/lens/server/TestLogResource.java > PRE-CREATION > lens-server/src/test/resources/lens-site.xml 3bcbf0e > lens-server/src/test/resources/log4j.properties e0f9daa > src/site/apt/admin/config.apt c5bf70c > src/site/apt/admin/deployment.apt 088a695 > > Diff: https://reviews.apache.org/r/36614/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.905s] > [INFO] Lens .............................................. SUCCESS [2.760s] > [INFO] Lens API .......................................... SUCCESS [20.260s] > [INFO] Lens API for server and extensions ................ SUCCESS [18.299s] > [INFO] Lens Cube ......................................... SUCCESS [4:15.156s] > [INFO] Lens DB storage ................................... SUCCESS [18.711s] > [INFO] Lens Query Library ................................ SUCCESS [16.986s] > [INFO] Lens Hive Driver .................................. SUCCESS [2:46.787s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [34.994s] > [INFO] Lens Server ....................................... SUCCESS [5:05.445s] > [INFO] Lens client ....................................... SUCCESS [39.535s] > [INFO] Lens CLI .......................................... SUCCESS [2:42.864s] > [INFO] Lens Examples ..................................... SUCCESS [8.755s] > [INFO] Lens Distribution ................................. SUCCESS [8.472s] > [INFO] Lens ML Lib ....................................... SUCCESS [1:19.460s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [1.987s] > [INFO] Lens Regression ................................... SUCCESS [9.775s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 19:12.983s > [INFO] Finished at: Mon Jul 20 10:45:55 UTC 2015 > [INFO] Final Memory: 192M/2361M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Amareshwari Sriramadasu > >
