----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31067/#review72571 -----------------------------------------------------------
lens-server/src/main/java/org/apache/lens/server/LensServer.java <https://reviews.apache.org/r/31067/#comment118621> not called in code anywhere? Shouldnt this be called in main? lens-server/src/test/java/org/apache/lens/server/TestLensServer.java <https://reviews.apache.org/r/31067/#comment118623> Can we directly use LensServerCong.get() ? lens-server/src/test/java/org/apache/lens/server/TestLensServer.java <https://reviews.apache.org/r/31067/#comment118622> Dont expose variables directly. Please use getters - Amareshwari Sriramadasu On Feb. 16, 2015, 5:29 a.m., Raju Bairishetti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31067/ > ----------------------------------------------------------- > > (Updated Feb. 16, 2015, 5:29 a.m.) > > > Review request for lens and Amareshwari Sriramadasu. > > > Bugs: LENS-304 > https://issues.apache.org/jira/browse/LENS-304 > > > Repository: lens > > > Description > ------- > > We have Lens UI http server different from Lens server. We should have > configuration option to know whether to bring up UI server or not. > > Changes done: > Providing a new config parameter for starting the ui server optional > > > Diffs > ----- > > > lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java > b4cef60 > lens-server/src/main/java/org/apache/lens/server/LensServer.java 5f42005 > lens-server/src/test/java/org/apache/lens/server/TestLensServer.java > PRE-CREATION > src/site/apt/admin/config.apt c5f3b2f > > Diff: https://reviews.apache.org/r/31067/diff/ > > > Testing > ------- > > Tested on local set up by starting the lens server. > > > Thanks, > > Raju Bairishetti > >
