----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11982/#review22199 -----------------------------------------------------------
Ship it! Ship It! - Jarek Cecho On June 19, 2013, 9:48 p.m., Venkat Ranganathan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11982/ > ----------------------------------------------------------- > > (Updated June 19, 2013, 9:48 p.m.) > > > Review request for Sqoop. > > > Description > ------- > > This addresses SQOOP-1017 > > I have split FrameworkManager into FrameworkManager and JobManager - one to > handle framework related activities and another to handle job management > activities > > > Diffs > ----- > > core/src/main/java/org/apache/sqoop/core/SqoopServer.java 84896d6 > core/src/main/java/org/apache/sqoop/framework/FrameworkManager.java 145a2c1 > core/src/main/java/org/apache/sqoop/framework/JobManager.java PRE-CREATION > server/src/main/java/org/apache/sqoop/handler/SubmissionRequestHandler.java > de9a24b > > Diff: https://reviews.apache.org/r/11982/diff/ > > > Testing > ------- > > Ran all tests and all pass. No new tests added as it was refactoring > existing functionality > > > Thanks, > > Venkat Ranganathan > >
