Sqoop
  1. Sqoop
  2. SQOOP-1211

Sqoop2: Derby repo: Sync maximal length of versions

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.99.3
    • Fix Version/s: 1.99.4
    • Component/s: None
    • Labels:
      None

      Description

      Derby repository currently defines the connector version in table SQC_CONNECTOR as 25 character long string whereas the framework version allows up to 64 character long strings (persisted in SQS_SYSTEM) table. It would be nice to get those two into sync.

      1. SQOOP-1211.patch
        4 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)" [ 12908 ]
          Jarek Jarcec Cecho made changes -
          Attachment SQOOP-1211.patch [ 12607163 ]
          Jarek Jarcec Cecho made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Sqoop QA bot added a comment -

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

          Overall: -1 due to an error

          ERROR: Patch does not add/modifny any test case
          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/161/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/12607163/SQOOP-1211.patch against branch sqoop2. Overall: -1 due to an error ERROR: Patch does not add/modifny any test case 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/161/console This message is automatically generated.
          Hide
          ASF subversion and git services added a comment -

          Commit caf4d824c1e8920324f28dd9de7d62cb2d87ca57 in branch refs/heads/sqoop2 from [~nrv]
          [ https://git-wip-us.apache.org/repos/asf?p=sqoop.git;h=caf4d82 ]

          SQOOP-1211: Sqoop2: Derby repo: Sync maximal length of versions
          (Jarek Jarcec Cecho via Venkat Ranganathan)

          Show
          ASF subversion and git services added a comment - Commit caf4d824c1e8920324f28dd9de7d62cb2d87ca57 in branch refs/heads/sqoop2 from [~nrv] [ https://git-wip-us.apache.org/repos/asf?p=sqoop.git;h=caf4d82 ] SQOOP-1211 : Sqoop2: Derby repo: Sync maximal length of versions (Jarek Jarcec Cecho via Venkat Ranganathan)
          Hide
          Venkat Ranganathan added a comment -

          Thanks Jarek Jarcec Cecho for your contribution. I ran full test and verification before committing it. Since this deals with upgrade situations lack of tests is understandable.

          Show
          Venkat Ranganathan added a comment - Thanks Jarek Jarcec Cecho for your contribution. I ran full test and verification before committing it. Since this deals with upgrade situations lack of tests is understandable.
          Hide
          Jarek Jarcec Cecho added a comment -

          Thank you Venkat Ranganathan for the review!

          Show
          Jarek Jarcec Cecho added a comment - Thank you Venkat Ranganathan for the review!
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Sqoop2-hadoop200 #518 (See https://builds.apache.org/job/Sqoop2-hadoop200/518/)
          SQOOP-1211: Sqoop2: Derby repo: Sync maximal length of versions (venkat: https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=caf4d824c1e8920324f28dd9de7d62cb2d87ca57)

          • repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java
          • repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoConstants.java
          • repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
          Show
          Hudson added a comment - SUCCESS: Integrated in Sqoop2-hadoop200 #518 (See https://builds.apache.org/job/Sqoop2-hadoop200/518/ ) SQOOP-1211 : Sqoop2: Derby repo: Sync maximal length of versions (venkat: https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=caf4d824c1e8920324f28dd9de7d62cb2d87ca57 ) repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoConstants.java repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Sqoop2-hadoop100 #585 (See https://builds.apache.org/job/Sqoop2-hadoop100/585/)
          SQOOP-1211: Sqoop2: Derby repo: Sync maximal length of versions (venkat: https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=caf4d824c1e8920324f28dd9de7d62cb2d87ca57)

          • repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
          • repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java
          • repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoConstants.java
          Show
          Hudson added a comment - SUCCESS: Integrated in Sqoop2-hadoop100 #585 (See https://builds.apache.org/job/Sqoop2-hadoop100/585/ ) SQOOP-1211 : Sqoop2: Derby repo: Sync maximal length of versions (venkat: https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=caf4d824c1e8920324f28dd9de7d62cb2d87ca57 ) repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoConstants.java
          Jarek Jarcec Cecho made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          11m 15s 1 Jarek Jarcec Cecho 07/Oct/13 16:24
          Patch Available Patch Available Resolved Resolved
          8d 1h 49m 1 Jarek Jarcec Cecho 15/Oct/13 18:14

            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