Sqoop
  1. Sqoop
  2. SQOOP-1183

Sqoop2: Add getConnector(String) to the client API

    Details

    • Type: Improvement Improvement
    • 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

      I would like to propose adding methods getConnector(String) and newConnection(String) to the Client API. My motivation is to make it easier where there is multiple connectors in the system as the cid can be on different servers different for the same connector while the name will be always uniquely pointing to the correct connector.

      1. SQOOP-1183.patch
        8 kB
        Jarek Jarcec Cecho

        Issue Links

          Activity

          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Sqoop2-hadoop100 #562 (See https://builds.apache.org/job/Sqoop2-hadoop100/562/)
          SQOOP-1183 Sqoop2. Add getConnector(String) to the client API (hshreedharan: https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=93270b804d174c96660aac3f5b5548fa0071761b)

          • client/src/main/java/org/apache/sqoop/client/ClientError.java
          • test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/imports/PartitionerTest.java
          • client/src/test/java/org/apache/sqoop/client/TestSqoopClient.java
          • client/src/main/java/org/apache/sqoop/client/SqoopClient.java
          • test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableImportTest.java
          • test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableExportTest.java
          Show
          Hudson added a comment - SUCCESS: Integrated in Sqoop2-hadoop100 #562 (See https://builds.apache.org/job/Sqoop2-hadoop100/562/ ) SQOOP-1183 Sqoop2. Add getConnector(String) to the client API (hshreedharan: https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=93270b804d174c96660aac3f5b5548fa0071761b ) client/src/main/java/org/apache/sqoop/client/ClientError.java test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/imports/PartitionerTest.java client/src/test/java/org/apache/sqoop/client/TestSqoopClient.java client/src/main/java/org/apache/sqoop/client/SqoopClient.java test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableImportTest.java test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableExportTest.java
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Sqoop2-hadoop200 #501 (See https://builds.apache.org/job/Sqoop2-hadoop200/501/)
          SQOOP-1183 Sqoop2. Add getConnector(String) to the client API (hshreedharan: https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=93270b804d174c96660aac3f5b5548fa0071761b)

          • test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableExportTest.java
          • test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/imports/PartitionerTest.java
          • client/src/test/java/org/apache/sqoop/client/TestSqoopClient.java
          • client/src/main/java/org/apache/sqoop/client/ClientError.java
          • client/src/main/java/org/apache/sqoop/client/SqoopClient.java
          • test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableImportTest.java
          Show
          Hudson added a comment - SUCCESS: Integrated in Sqoop2-hadoop200 #501 (See https://builds.apache.org/job/Sqoop2-hadoop200/501/ ) SQOOP-1183 Sqoop2. Add getConnector(String) to the client API (hshreedharan: https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=93270b804d174c96660aac3f5b5548fa0071761b ) test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableExportTest.java test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/imports/PartitionerTest.java client/src/test/java/org/apache/sqoop/client/TestSqoopClient.java client/src/main/java/org/apache/sqoop/client/ClientError.java client/src/main/java/org/apache/sqoop/client/SqoopClient.java test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableImportTest.java
          Hide
          Hari Shreedharan added a comment -

          Committed!

          Show
          Hari Shreedharan added a comment - Committed!
          Hide
          ASF subversion and git services added a comment -

          Commit 93270b804d174c96660aac3f5b5548fa0071761b in branch refs/heads/sqoop2 from Hari Shreedharan
          [ https://git-wip-us.apache.org/repos/asf?p=sqoop.git;h=93270b8 ]

          SQOOP-1183 Sqoop2. Add getConnector(String) to the client API

          (Jarek Jarcec Cecho via Hari Shreedharan)

          Show
          ASF subversion and git services added a comment - Commit 93270b804d174c96660aac3f5b5548fa0071761b in branch refs/heads/sqoop2 from Hari Shreedharan [ https://git-wip-us.apache.org/repos/asf?p=sqoop.git;h=93270b8 ] SQOOP-1183 Sqoop2. Add getConnector(String) to the client API (Jarek Jarcec Cecho via Hari Shreedharan)
          Hide
          Sqoop QA bot added a comment -

          Here are the results of testing the latest attachment
          https://issues.apache.org/jira/secure/attachment/12599813/SQOOP-1183.patch against branch sqoop2.

          Overall: +1 all checks pass

          SUCCESS: Clean was successful
          SUCCESS: Patch applied correctly
          SUCCESS: Patch compiled
          SUCCESS: All tests passed

          Console output: https://builds.apache.org/job/PreCommit-SQOOP-Build/129/console

          This message is automatically generated.

          Show
          Sqoop QA bot added a comment - Here are the results of testing the latest attachment https://issues.apache.org/jira/secure/attachment/12599813/SQOOP-1183.patch against branch sqoop2. Overall: +1 all checks pass SUCCESS: Clean was successful SUCCESS: Patch applied correctly SUCCESS: Patch compiled SUCCESS: All tests passed Console output: https://builds.apache.org/job/PreCommit-SQOOP-Build/129/console This message is automatically generated.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development