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

Switch to Threaded View
Sqoop, mail # dev - Review Request:  Sqoop2: Add cloning ability to model classes


Copy link to this message
-
Re: Review Request:  Sqoop2: Add cloning ability to model classes
rj.vasanthkumar@... 2013-05-06, 18:38

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

(Updated May 6, 2013, 6:38 p.m.)
Review request for Sqoop.
Changes
-------

Implemented Jarek's suggestion. Kindly review.
Description (updated)
-------

Cloning Model classes by introduce clone method in model classes.
This addresses bug SQOOP-995.
    https://issues.apache.org/jira/browse/SQOOP-995
Diffs (updated)
-----

  common/src/main/java/org/apache/sqoop/model/MClonable.java PRE-CREATION
  common/src/main/java/org/apache/sqoop/model/MConnection.java c31eafd
  common/src/main/java/org/apache/sqoop/model/MConnectionForms.java d289a02
  common/src/main/java/org/apache/sqoop/model/MConnector.java 9207c62
  common/src/main/java/org/apache/sqoop/model/MEnumInput.java 12705a6
  common/src/main/java/org/apache/sqoop/model/MForm.java 76998dd
  common/src/main/java/org/apache/sqoop/model/MFormList.java 3bf508d
  common/src/main/java/org/apache/sqoop/model/MFramework.java f5252ef
  common/src/main/java/org/apache/sqoop/model/MInput.java 7d6215f
  common/src/main/java/org/apache/sqoop/model/MIntegerInput.java d23ac31
  common/src/main/java/org/apache/sqoop/model/MJob.java 5b50bfd
  common/src/main/java/org/apache/sqoop/model/MJobForms.java bce646f
  common/src/main/java/org/apache/sqoop/model/MMapInput.java 704c1f8
  common/src/main/java/org/apache/sqoop/model/MStringInput.java a437a25
  common/src/test/java/org/apache/sqoop/model/TestMConnection.java c3469ff
  common/src/test/java/org/apache/sqoop/model/TestMConnector.java 716b124
  common/src/test/java/org/apache/sqoop/model/TestMJob.java dbc791e

Diff: https://reviews.apache.org/r/10886/diff/
Testing (updated)
-------

Updated related test classes.
Thanks,

vasanthkumar