----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31067/#review72684 -----------------------------------------------------------
lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java <https://reviews.apache.org/r/31067/#comment118752> Shall we call the config name lens.server.ui.enabled ? .start does not make sense. lens-server/src/main/java/org/apache/lens/server/LensServer.java <https://reviews.apache.org/r/31067/#comment118753> call this only isUIServerEnabled - Amareshwari Sriramadasu On Feb. 17, 2015, 2:47 a.m., Raju Bairishetti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31067/ > ----------------------------------------------------------- > > (Updated Feb. 17, 2015, 2:47 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 > >
