Sqoop
  1. Sqoop
  2. SQOOP-995

Sqoop2: Add cloning ability to model classes

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.99.2
    • Fix Version/s: 1.99.3
    • Component/s: None
    • Labels:
      None

      Description

      We've introduced quick "cloneForms" method during SQOOP-652. I think that we should refactore the method by introducing cloning ability to the model classes themselves.

      1. SQOOP-995.patch
        21 kB
        Vasanth kumar RJ
      2. SQOOP-995.patch
        11 kB
        Vasanth kumar RJ

        Issue Links

          Activity

          Jarek Jarcec Cecho created issue -
          Jarek Jarcec Cecho made changes -
          Field Original Value New Value
          Fix Version/s 2.0.0 [ 12319272 ]
          Jarek Jarcec Cecho made changes -
          Link This issue blocks SQOOP-1019 [ SQOOP-1019 ]
          Hide
          Vasanth kumar RJ added a comment -

          Hi Jarek,

          For cloning the model classes containing forms, need to preserve value of form input as well?

          Cloning by creating copy constructor. In case if preserve form input's value, can we have cloning with and without form input value?

          Kindly suggest.

          Thanks,
          Vasanth kumar

          Show
          Vasanth kumar RJ added a comment - Hi Jarek, For cloning the model classes containing forms, need to preserve value of form input as well? Cloning by creating copy constructor. In case if preserve form input's value, can we have cloning with and without form input value? Kindly suggest. Thanks, Vasanth kumar
          Vasanth kumar RJ made changes -
          Attachment SQOOP-995.patch [ 12581415 ]
          Vasanth kumar RJ made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Assignee Vasanth kumar RJ [ vasanthkumar ]
          Jarek Jarcec Cecho made changes -
          Remote Link This issue links to "Review board (Web Link)" [ 12190 ]
          Hide
          Jarek Jarcec Cecho added a comment -

          Hi Vasanth kumar RJ,
          thank you for picking up this one. I've left few comments on the review board. I think that copying values should be configurable because we might need both behaviorism (e.g. sometimes create copy with values, sometimes without).

          Jarcec

          Show
          Jarek Jarcec Cecho added a comment - Hi Vasanth kumar RJ , thank you for picking up this one. I've left few comments on the review board. I think that copying values should be configurable because we might need both behaviorism (e.g. sometimes create copy with values, sometimes without). Jarcec
          Vasanth kumar RJ made changes -
          Attachment SQOOP-995.patch [ 12584362 ]
          Show
          Jarek Jarcec Cecho added a comment - The patch is in: https://git-wip-us.apache.org/repos/asf?p=sqoop.git;a=commit;h=94c91efd3bfc87123ec11589aa32812a23101d3e Thank you Vasanth! Jarcec
          Jarek Jarcec Cecho made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Fix Version/s 1.99.3 [ 12324298 ]
          Fix Version/s 2.0.0 [ 12319272 ]
          Resolution Fixed [ 1 ]
          Hide
          Hudson added a comment -

          Integrated in Sqoop2-hadoop100 #289 (See https://builds.apache.org/job/Sqoop2-hadoop100/289/)
          SQOOP-995: Sqoop2: Add cloning ability to model classes (Revision 94c91efd3bfc87123ec11589aa32812a23101d3e)

          Result = SUCCESS
          jarcec : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=94c91efd3bfc87123ec11589aa32812a23101d3e
          Files :

          • common/src/test/java/org/apache/sqoop/model/TestMJob.java
          • common/src/main/java/org/apache/sqoop/model/MFramework.java
          • common/src/main/java/org/apache/sqoop/model/MStringInput.java
          • common/src/main/java/org/apache/sqoop/model/MEnumInput.java
          • common/src/main/java/org/apache/sqoop/model/MClonable.java
          • common/src/main/java/org/apache/sqoop/model/MIntegerInput.java
          • common/src/main/java/org/apache/sqoop/model/MJob.java
          • common/src/test/java/org/apache/sqoop/model/TestMConnection.java
          • common/src/main/java/org/apache/sqoop/model/MFormList.java
          • common/src/main/java/org/apache/sqoop/model/MMapInput.java
          • common/src/main/java/org/apache/sqoop/model/MConnectionForms.java
          • common/src/main/java/org/apache/sqoop/model/MForm.java
          • common/src/main/java/org/apache/sqoop/model/MJobForms.java
          • common/src/main/java/org/apache/sqoop/model/MInput.java
          • common/src/test/java/org/apache/sqoop/model/TestMConnector.java
          • common/src/main/java/org/apache/sqoop/model/MConnection.java
          • common/src/main/java/org/apache/sqoop/model/MConnector.java
          Show
          Hudson added a comment - Integrated in Sqoop2-hadoop100 #289 (See https://builds.apache.org/job/Sqoop2-hadoop100/289/ ) SQOOP-995 : Sqoop2: Add cloning ability to model classes (Revision 94c91efd3bfc87123ec11589aa32812a23101d3e) Result = SUCCESS jarcec : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=94c91efd3bfc87123ec11589aa32812a23101d3e Files : common/src/test/java/org/apache/sqoop/model/TestMJob.java common/src/main/java/org/apache/sqoop/model/MFramework.java common/src/main/java/org/apache/sqoop/model/MStringInput.java common/src/main/java/org/apache/sqoop/model/MEnumInput.java common/src/main/java/org/apache/sqoop/model/MClonable.java common/src/main/java/org/apache/sqoop/model/MIntegerInput.java common/src/main/java/org/apache/sqoop/model/MJob.java common/src/test/java/org/apache/sqoop/model/TestMConnection.java common/src/main/java/org/apache/sqoop/model/MFormList.java common/src/main/java/org/apache/sqoop/model/MMapInput.java common/src/main/java/org/apache/sqoop/model/MConnectionForms.java common/src/main/java/org/apache/sqoop/model/MForm.java common/src/main/java/org/apache/sqoop/model/MJobForms.java common/src/main/java/org/apache/sqoop/model/MInput.java common/src/test/java/org/apache/sqoop/model/TestMConnector.java common/src/main/java/org/apache/sqoop/model/MConnection.java common/src/main/java/org/apache/sqoop/model/MConnector.java
          Hide
          Hudson added a comment -

          Integrated in Sqoop2-hadoop200 #293 (See https://builds.apache.org/job/Sqoop2-hadoop200/293/)
          SQOOP-995: Sqoop2: Add cloning ability to model classes (Revision 94c91efd3bfc87123ec11589aa32812a23101d3e)

          Result = SUCCESS
          jarcec : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=94c91efd3bfc87123ec11589aa32812a23101d3e
          Files :

          • common/src/main/java/org/apache/sqoop/model/MStringInput.java
          • common/src/main/java/org/apache/sqoop/model/MConnector.java
          • common/src/main/java/org/apache/sqoop/model/MFormList.java
          • common/src/main/java/org/apache/sqoop/model/MJob.java
          • common/src/main/java/org/apache/sqoop/model/MClonable.java
          • common/src/main/java/org/apache/sqoop/model/MJobForms.java
          • common/src/test/java/org/apache/sqoop/model/TestMConnector.java
          • common/src/main/java/org/apache/sqoop/model/MConnection.java
          • common/src/main/java/org/apache/sqoop/model/MFramework.java
          • common/src/test/java/org/apache/sqoop/model/TestMJob.java
          • common/src/main/java/org/apache/sqoop/model/MEnumInput.java
          • common/src/main/java/org/apache/sqoop/model/MInput.java
          • common/src/main/java/org/apache/sqoop/model/MIntegerInput.java
          • common/src/test/java/org/apache/sqoop/model/TestMConnection.java
          • common/src/main/java/org/apache/sqoop/model/MMapInput.java
          • common/src/main/java/org/apache/sqoop/model/MForm.java
          • common/src/main/java/org/apache/sqoop/model/MConnectionForms.java
          Show
          Hudson added a comment - Integrated in Sqoop2-hadoop200 #293 (See https://builds.apache.org/job/Sqoop2-hadoop200/293/ ) SQOOP-995 : Sqoop2: Add cloning ability to model classes (Revision 94c91efd3bfc87123ec11589aa32812a23101d3e) Result = SUCCESS jarcec : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=94c91efd3bfc87123ec11589aa32812a23101d3e Files : common/src/main/java/org/apache/sqoop/model/MStringInput.java common/src/main/java/org/apache/sqoop/model/MConnector.java common/src/main/java/org/apache/sqoop/model/MFormList.java common/src/main/java/org/apache/sqoop/model/MJob.java common/src/main/java/org/apache/sqoop/model/MClonable.java common/src/main/java/org/apache/sqoop/model/MJobForms.java common/src/test/java/org/apache/sqoop/model/TestMConnector.java common/src/main/java/org/apache/sqoop/model/MConnection.java common/src/main/java/org/apache/sqoop/model/MFramework.java common/src/test/java/org/apache/sqoop/model/TestMJob.java common/src/main/java/org/apache/sqoop/model/MEnumInput.java common/src/main/java/org/apache/sqoop/model/MInput.java common/src/main/java/org/apache/sqoop/model/MIntegerInput.java common/src/test/java/org/apache/sqoop/model/TestMConnection.java common/src/main/java/org/apache/sqoop/model/MMapInput.java common/src/main/java/org/apache/sqoop/model/MForm.java common/src/main/java/org/apache/sqoop/model/MConnectionForms.java
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          12d 16h 42m 1 Vasanth kumar RJ 01/May/13 22:02
          Patch Available Patch Available Resolved Resolved
          21d 11h 15m 1 Jarek Jarcec Cecho 23/May/13 09:18

            People

            • Assignee:
              Vasanth kumar RJ
              Reporter:
              Jarek Jarcec Cecho
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development