Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Sqoop >> mail # dev >> Re: Review Request: Fix for SQOOP-107


+
Shruti Joshi 2013-05-30, 13:12
+
Venkat Ranganathan 2013-05-30, 06:07
+
Venkat Ranganathan 2013-05-30, 18:25
+
Abraham Elmahrek 2013-05-31, 20:30
+
Venkat Ranganathan 2013-06-01, 00:07
+
Abraham Elmahrek 2013-05-31, 20:33
+
Jarek Cecho 2013-06-10, 15:36
+
Venkat Ranganathan 2013-06-10, 16:06
+
Venkat Ranganathan 2013-06-11, 18:41
+
Venkat Ranganathan 2013-06-12, 17:23
+
Venkat Ranganathan 2013-06-13, 05:46
+
Abraham Elmahrek 2013-06-13, 19:00
+
Venkat Ranganathan 2013-06-13, 20:32
+
Jarek Cecho 2013-06-13, 20:35
+
Abraham Elmahrek 2013-06-13, 01:41
+
Venkat Ranganathan 2013-06-13, 05:44
+
Shruti Joshi 2013-06-03, 13:06
+
Shruti Joshi 2013-06-03, 13:04
+
Jarek Cecho 2013-06-08, 16:19
+
Shruti Joshi 2013-06-19, 10:12
+
Shruti Joshi 2013-06-19, 10:17
+
Shruti Joshi 2013-06-20, 08:50
+
Jarek Cecho 2013-06-20, 16:20
+
Shruti Joshi 2013-06-21, 04:12
+
Jarek Cecho 2013-06-19, 20:50
+
Shruti Joshi 2013-06-21, 03:40
Copy link to this message
-
Review Request: Fix for SQOOP-1017

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

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

+
Jarek Cecho 2013-06-20, 22:02