----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29036/#review65090 -----------------------------------------------------------
Ship it! Ship It! - Jaideep dhok On Dec. 15, 2014, 3:06 p.m., Jothi Padmanabhan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29036/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2014, 3:06 p.m.) > > > Review request for lens. > > > Bugs: lens-104 > https://issues.apache.org/jira/browse/lens-104 > > > Repository: lens > > > Description > ------- > > Lens-104. Add session service to server-api > > > Diffs > ----- > > lens-ml-lib/src/test/java/org/apache/lens/ml/TestMLResource.java a0abe93 > > lens-server-api/src/main/java/org/apache/lens/server/api/session/SessionService.java > PRE-CREATION > > lens-server/src/main/java/org/apache/lens/server/session/HiveSessionService.java > f45fe3c > > lens-server/src/main/java/org/apache/lens/server/session/SessionResource.java > 4d4bb53 > lens-server/src/main/java/org/apache/lens/server/ui/SessionUIResource.java > 29dc669 > > lens-server/src/test/java/org/apache/lens/server/session/TestSessionResource.java > 10f65b2 > > Diff: https://reviews.apache.org/r/29036/diff/ > > > Testing > ------- > > Changes are in interface, no new tests added. Also, did not change the > TestSessionUI and TestServerRestart as they seem to be testing the internals > of the HiveSessionService implementation > > > Thanks, > > Jothi Padmanabhan > >
