You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lens.apache.org by Amareshwari Sriramadasu <am...@apache.org> on 2016/07/01 05:31:10 UTC

Re: Review Request 45099: LENS-991: Seeing multiple close calls and failures of connection after LENS-853

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45099/#review140306
-----------------------------------------------------------


Ship it!




Ship It!

- Amareshwari Sriramadasu


On June 28, 2016, 10:59 a.m., Raju Bairishetti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45099/
> -----------------------------------------------------------
> 
> (Updated June 28, 2016, 10:59 a.m.)
> 
> 
> Review request for lens, Amareshwari Sriramadasu and Rajat Khandelwal.
> 
> 
> Bugs: lens-991
>     https://issues.apache.org/jira/browse/lens-991
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Shutdown hook was getting called for each lens **command bean** whenver we do ctrl+c on cli.
> 
> 
> Diffs
> -----
> 
>   checkstyle/src/main/resources/checkstyle.xml 0a37d01 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/BaseLensCommand.java 9eb73b3 
>   lens-cli/src/test/java/org/apache/lens/cli/TestLensConnectionCliCommands.java d1ec94e 
>   lens-cli/src/test/java/org/apache/lens/cli/TestLensCubeCommands.java f53ed6b 
>   lens-cli/src/test/java/org/apache/lens/cli/TestLensDatabaseCommands.java 705aace 
>   lens-cli/src/test/java/org/apache/lens/cli/TestLensNativeTableCommands.java 80dd0d1 
>   lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java 4d8341a 
>   lens-client/src/test/java/org/apache/lens/client/TestLensClient.java ea464d5 
> 
> Diff: https://reviews.apache.org/r/45099/diff/
> 
> 
> Testing
> -------
> 
> **Before applying patch:**
> 
> lens-shell>show cubes
> No cube found
> lens-shell>show params
> ...
> lens-shell>Exception in thread "Thread-3" java.lang.IllegalStateException: Unable to close lens connection with params LensConnectionParams{dbName='default', baseUrl='http://0.0.0.0:9999/lensapi', user=anonymous, lensConfs={}, lensVars={}, sessionVars={}}
> 	at org.apache.lens.client.LensConnection.close(LensConnection.java:206)
> 	at org.apache.lens.client.LensClient.closeConnection(LensClient.java:326)
> 	at org.apache.lens.cli.commands.BaseLensCommand.closeClientConnection(BaseLensCommand.java:92)
> 	at org.apache.lens.cli.commands.BaseLensCommand$2.run(BaseLensCommand.java:81)
> LZDLocalAdmins-MacBook-Pro-2:client rajubairishetti$
>     
>     
>     
> **After aplying patch:**
> 
> lens-shell>show cubes
> No cube found
> lens-shell>show params
> ...
> lens-shell>LZDLocalAdmins-MacBook-Pro-2:client rajubairishetti$
> 
> 
> All tests are passing except TestUnionQueries from lens-cube. Jira is already present for that. 
> 
> [INFO] Reactor Summary:
> [INFO]
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [3.835s]
> [INFO] Lens .............................................. SUCCESS [4.721s]
> [INFO] Lens API .......................................... SUCCESS [26.082s]
> [INFO] Lens API for server and extensions ................ SUCCESS [22.151s]
> [INFO] Lens Cube ......................................... SUCCESS [9:31.256s]
> [INFO] Lens DB storage ................................... SUCCESS [15.318s]
> [INFO] Lens Query Library ................................ SUCCESS [13.582s]
> [INFO] Lens Hive Driver .................................. SUCCESS [1:21.532s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [30.481s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [18.666s]
> [INFO] Lens Server ....................................... SUCCESS [13:40.026s]
> [INFO] Lens client ....................................... SUCCESS [1:31.229s]
> [INFO] Lens CLI .......................................... SUCCESS [1:17.334s]
> [INFO] Lens Examples ..................................... SUCCESS [8.929s]
> [INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.888s]
> [INFO] Lens Distribution ................................. SUCCESS [11.928s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:08.603s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.754s]
> [INFO] Lens Regression ................................... SUCCESS [15.555s]
> [INFO] Lens UI ........................................... SUCCESS [0.997s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 31:26.443s
> [INFO] Finished at: Tue Jun 28 10:53:01 SGT 2016
> [INFO] Final Memory: 214M/1170M
> [INFO] ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Raju Bairishetti
> 
>