Sqoop
  1. Sqoop
  2. SQOOP-799

Sqoop2: Change SqoopConnector interface to abstract class

    Details

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

      Description

      I would suggest to convert SqoopConnector interface to abstract class similarly as we've done all over the place.

      1. bugSQOOP-799.patch
        3 kB
        Jarek Jarcec Cecho

        Issue Links

          Activity

          Jarek Jarcec Cecho created issue -
          Jarek Jarcec Cecho made changes -
          Field Original Value New Value
          Remote Link This issue links to "Review board (Web Link)" [ 11926 ]
          Jarek Jarcec Cecho made changes -
          Attachment bugSQOOP-799.patch [ 12562014 ]
          Jarek Jarcec Cecho made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Bilung Lee added a comment -

          Thanks. Patch is in.

          Show
          Bilung Lee added a comment - Thanks. Patch is in.
          Bilung Lee made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Hudson added a comment -

          Integrated in Sqoop2-hadoop200 #46 (See https://builds.apache.org/job/Sqoop2-hadoop200/46/)
          SQOOP-799 Change SqoopConnector interface to abstract class (Revision f55b132035f042f24a1052da56da570fc91bae7f)

          Result = UNSTABLE
          blee : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=f55b132035f042f24a1052da56da570fc91bae7f
          Files :

          • connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcConnector.java
          • spi/src/main/java/org/apache/sqoop/connector/spi/SqoopConnector.java
          Show
          Hudson added a comment - Integrated in Sqoop2-hadoop200 #46 (See https://builds.apache.org/job/Sqoop2-hadoop200/46/ ) SQOOP-799 Change SqoopConnector interface to abstract class (Revision f55b132035f042f24a1052da56da570fc91bae7f) Result = UNSTABLE blee : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=f55b132035f042f24a1052da56da570fc91bae7f Files : connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcConnector.java spi/src/main/java/org/apache/sqoop/connector/spi/SqoopConnector.java
          Hide
          Hudson added a comment -

          Integrated in Sqoop2-hadoop100 #45 (See https://builds.apache.org/job/Sqoop2-hadoop100/45/)
          SQOOP-799 Change SqoopConnector interface to abstract class (Revision f55b132035f042f24a1052da56da570fc91bae7f)

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

          • spi/src/main/java/org/apache/sqoop/connector/spi/SqoopConnector.java
          • connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcConnector.java
          Show
          Hudson added a comment - Integrated in Sqoop2-hadoop100 #45 (See https://builds.apache.org/job/Sqoop2-hadoop100/45/ ) SQOOP-799 Change SqoopConnector interface to abstract class (Revision f55b132035f042f24a1052da56da570fc91bae7f) Result = SUCCESS blee : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=f55b132035f042f24a1052da56da570fc91bae7f Files : spi/src/main/java/org/apache/sqoop/connector/spi/SqoopConnector.java connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcConnector.java
          Hide
          Jarek Jarcec Cecho added a comment -

          The UNSTABLE for hadoop200 seems as an temporal glitch. It's passing locally for me and rerun also cleared the issue on jenkins.

          Show
          Jarek Jarcec Cecho added a comment - The UNSTABLE for hadoop200 seems as an temporal glitch. It's passing locally for me and rerun also cleared the issue on jenkins.
          Jarek Jarcec Cecho made changes -
          Link This issue is duplicated by SQOOP-763 [ SQOOP-763 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development