----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46938/#review131670 -----------------------------------------------------------
lens-regression/src/test/java/org/apache/lens/regression/client/ITQueryApiTests.java (line 325) <https://reviews.apache.org/r/46938/#comment195720> Remove sys outs ? lens-regression/src/test/java/org/apache/lens/regression/config/ITServerConfigTests.java (line 138) <https://reviews.apache.org/r/46938/#comment195709> Instead of String, Can we use Boolean ? lens-regression/src/test/java/org/apache/lens/regression/config/ITServerConfigTests.java (line 514) <https://reviews.apache.org/r/46938/#comment195714> Should be 404 error ? lens-regression/src/test/java/org/apache/lens/regression/config/ITServerConfigTests.java (line 578) <https://reviews.apache.org/r/46938/#comment195718> You can create a static final Integer constant for 429 in this case and reuse - private final Integer HTTP_RESPONSE_CODE_FOR_TOO_MANY_REQUESTS = 429; lens-regression/src/test/java/org/apache/lens/regression/config/ITServerConfigTests.java (line 582) <https://reviews.apache.org/r/46938/#comment195719> You can simply do sHelper.closeNewSession(sessionList1.remove(1)); instead of sHelper.closeNewSession(sessionList1.get(1)); sessionList1.remove(1); - Deepak Barr On May 4, 2016, 7:35 a.m., Archana H wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46938/ > ----------------------------------------------------------- > > (Updated May 4, 2016, 7:35 a.m.) > > > Review request for lens, Deepak Barr, Rajat Khandelwal, and Puneet Gupta. > > > Repository: lens > > > Description > ------- > > Adding server configuration tests to regression > > > Diffs > ----- > > > lens-regression/src/main/java/org/apache/lens/regression/core/constants/DriverConfig.java > 5a2df11 > > lens-regression/src/main/java/org/apache/lens/regression/core/constants/QueryInventory.java > 869a96b > > lens-regression/src/main/java/org/apache/lens/regression/core/helpers/LensServerHelper.java > 2f8b9c0 > > lens-regression/src/main/java/org/apache/lens/regression/core/helpers/QueryHelper.java > 2b4a750 > > lens-regression/src/main/java/org/apache/lens/regression/core/helpers/SessionHelper.java > 4918882 > > lens-regression/src/main/java/org/apache/lens/regression/util/AssertUtil.java > ea36858 > lens-regression/src/main/java/org/apache/lens/regression/util/Util.java > 6c3d234 > > lens-regression/src/test/java/org/apache/lens/regression/client/ITListQueryTest.java > 60cf26c > > lens-regression/src/test/java/org/apache/lens/regression/client/ITPreparedQueryTests.java > bf40c74 > > lens-regression/src/test/java/org/apache/lens/regression/client/ITQueryApiTests.java > e16b220 > > lens-regression/src/test/java/org/apache/lens/regression/client/KillQueryTests.java > 8aabe2d > > lens-regression/src/test/java/org/apache/lens/regression/client/SessionResourceTests.java > 62dd0ba > > lens-regression/src/test/java/org/apache/lens/regression/config/ITServerConfigTests.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/46938/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.981s] > [INFO] Lens .............................................. SUCCESS [3.316s] > [INFO] Lens API .......................................... SUCCESS [19.095s] > [INFO] Lens API for server and extensions ................ SUCCESS [15.684s] > [INFO] Lens Cube ......................................... SUCCESS [6:48.240s] > [INFO] Lens DB storage ................................... SUCCESS [15.062s] > [INFO] Lens Query Library ................................ SUCCESS [12.688s] > [INFO] Lens Hive Driver .................................. SUCCESS [2:25.271s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [31.631s] > [INFO] Lens Elastic Search Driver ........................ SUCCESS [12.114s] > [INFO] Lens Server ....................................... SUCCESS > [13:32.537s] > [INFO] Lens client ....................................... SUCCESS [1:42.565s] > [INFO] Lens CLI .......................................... SUCCESS [2:07.638s] > [INFO] Lens Examples ..................................... SUCCESS [8.887s] > [INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.678s] > [INFO] Lens Distribution ................................. SUCCESS [9.547s] > [INFO] Lens ML Lib ....................................... SUCCESS [1:06.594s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [1.511s] > [INFO] Lens Regression ................................... SUCCESS [9.505s] > [INFO] Lens UI ........................................... SUCCESS [54.924s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 31:00.347s > [INFO] Finished at: Tue May 03 14:59:52 UTC 2016 > [INFO] Final Memory: 182M/2644M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Archana H > >
