Sqoop
  1. Sqoop
  2. SQOOP-820

Escape table name in export job only if it's required by connector

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4.3
    • Component/s: None
    • Labels:
      None

      Description

      During investigation on SQOOP-818 I've noticed that automatically escaping table name when saving to mapreduce configuration introduced in SQOOP-601 might cause issues to third party connectors that are expecting that the name won't be ever escaped.

      1. SQOOP-820.patch
        3 kB
        Jarek Jarcec Cecho

        Issue Links

          Activity

          Hide
          Cheolsoo Park added a comment -

          +1.

          Committed to trunk. Thanks Jarcec!

          Show
          Cheolsoo Park added a comment - +1. Committed to trunk. Thanks Jarcec!
          Hide
          Hudson added a comment -

          Integrated in Sqoop-ant-jdk-1.6-hadoop20 #498 (See https://builds.apache.org/job/Sqoop-ant-jdk-1.6-hadoop20/498/)
          SQOOP-820: Escape table name in export job only if it's required by connector (Revision b02ddc39757771fb387c2688ccd0cee74cf6032c)

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

          • src/java/org/apache/sqoop/manager/PostgresqlManager.java
          • src/java/org/apache/sqoop/manager/ConnManager.java
          • src/java/org/apache/sqoop/mapreduce/JdbcExportJob.java
          • src/java/org/apache/sqoop/manager/SQLServerManager.java
          Show
          Hudson added a comment - Integrated in Sqoop-ant-jdk-1.6-hadoop20 #498 (See https://builds.apache.org/job/Sqoop-ant-jdk-1.6-hadoop20/498/ ) SQOOP-820 : Escape table name in export job only if it's required by connector (Revision b02ddc39757771fb387c2688ccd0cee74cf6032c) Result = SUCCESS cheolsoo : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=b02ddc39757771fb387c2688ccd0cee74cf6032c Files : src/java/org/apache/sqoop/manager/PostgresqlManager.java src/java/org/apache/sqoop/manager/ConnManager.java src/java/org/apache/sqoop/mapreduce/JdbcExportJob.java src/java/org/apache/sqoop/manager/SQLServerManager.java
          Hide
          Hudson added a comment -

          Integrated in Sqoop-ant-jdk-1.6-hadoop200 #483 (See https://builds.apache.org/job/Sqoop-ant-jdk-1.6-hadoop200/483/)
          SQOOP-820: Escape table name in export job only if it's required by connector (Revision b02ddc39757771fb387c2688ccd0cee74cf6032c)

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

          • src/java/org/apache/sqoop/mapreduce/JdbcExportJob.java
          • src/java/org/apache/sqoop/manager/SQLServerManager.java
          • src/java/org/apache/sqoop/manager/PostgresqlManager.java
          • src/java/org/apache/sqoop/manager/ConnManager.java
          Show
          Hudson added a comment - Integrated in Sqoop-ant-jdk-1.6-hadoop200 #483 (See https://builds.apache.org/job/Sqoop-ant-jdk-1.6-hadoop200/483/ ) SQOOP-820 : Escape table name in export job only if it's required by connector (Revision b02ddc39757771fb387c2688ccd0cee74cf6032c) Result = SUCCESS cheolsoo : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=b02ddc39757771fb387c2688ccd0cee74cf6032c Files : src/java/org/apache/sqoop/mapreduce/JdbcExportJob.java src/java/org/apache/sqoop/manager/SQLServerManager.java src/java/org/apache/sqoop/manager/PostgresqlManager.java src/java/org/apache/sqoop/manager/ConnManager.java
          Hide
          Hudson added a comment -

          Integrated in Sqoop-ant-jdk-1.6-hadoop23 #693 (See https://builds.apache.org/job/Sqoop-ant-jdk-1.6-hadoop23/693/)
          SQOOP-820: Escape table name in export job only if it's required by connector (Revision b02ddc39757771fb387c2688ccd0cee74cf6032c)

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

          • src/java/org/apache/sqoop/manager/SQLServerManager.java
          • src/java/org/apache/sqoop/manager/ConnManager.java
          • src/java/org/apache/sqoop/mapreduce/JdbcExportJob.java
          • src/java/org/apache/sqoop/manager/PostgresqlManager.java
          Show
          Hudson added a comment - Integrated in Sqoop-ant-jdk-1.6-hadoop23 #693 (See https://builds.apache.org/job/Sqoop-ant-jdk-1.6-hadoop23/693/ ) SQOOP-820 : Escape table name in export job only if it's required by connector (Revision b02ddc39757771fb387c2688ccd0cee74cf6032c) Result = SUCCESS cheolsoo : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=b02ddc39757771fb387c2688ccd0cee74cf6032c Files : src/java/org/apache/sqoop/manager/SQLServerManager.java src/java/org/apache/sqoop/manager/ConnManager.java src/java/org/apache/sqoop/mapreduce/JdbcExportJob.java src/java/org/apache/sqoop/manager/PostgresqlManager.java
          Hide
          Hudson added a comment -

          Integrated in Sqoop-ant-jdk-1.6-hadoop100 #487 (See https://builds.apache.org/job/Sqoop-ant-jdk-1.6-hadoop100/487/)
          SQOOP-820: Escape table name in export job only if it's required by connector (Revision b02ddc39757771fb387c2688ccd0cee74cf6032c)

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

          • src/java/org/apache/sqoop/manager/SQLServerManager.java
          • src/java/org/apache/sqoop/manager/ConnManager.java
          • src/java/org/apache/sqoop/mapreduce/JdbcExportJob.java
          • src/java/org/apache/sqoop/manager/PostgresqlManager.java
          Show
          Hudson added a comment - Integrated in Sqoop-ant-jdk-1.6-hadoop100 #487 (See https://builds.apache.org/job/Sqoop-ant-jdk-1.6-hadoop100/487/ ) SQOOP-820 : Escape table name in export job only if it's required by connector (Revision b02ddc39757771fb387c2688ccd0cee74cf6032c) Result = SUCCESS cheolsoo : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=b02ddc39757771fb387c2688ccd0cee74cf6032c Files : src/java/org/apache/sqoop/manager/SQLServerManager.java src/java/org/apache/sqoop/manager/ConnManager.java src/java/org/apache/sqoop/mapreduce/JdbcExportJob.java src/java/org/apache/sqoop/manager/PostgresqlManager.java

            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