Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Sqoop, mail # dev - Review Request 14351: SQOOP-1208 Sqoop2: job ID column is too small


+
Abraham Elmahrek 2013-09-26, 20:02
+
Jarek Cecho 2013-09-26, 21:13
+
Abraham Elmahrek 2013-09-27, 23:02
+
Jarek Cecho 2013-10-01, 02:13
+
Abraham Elmahrek 2013-10-01, 02:41
+
Abraham Elmahrek 2013-10-01, 02:45
Copy link to this message
-
Re: Review Request 14351: SQOOP-1208 Sqoop2: job ID column is too small
Abraham Elmahrek 2013-10-01, 07:00

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14351/
-----------------------------------------------------------

(Updated Oct. 1, 2013, 6:59 a.m.)
Review request for Sqoop, Hari Shreedharan and Jarek Cecho.
Bugs: SQOOP-1208
    https://issues.apache.org/jira/browse/SQOOP-1208
Repository: sqoop-sqoop2
Description
-------

commit eb0e73f08c80859c979aac17cf8df8522b43368a
Author: Abraham Elmahrek <[EMAIL PROTECTED]>
Date:   Thu Sep 26 12:59:42 2013 -0700

    SQOOP-1208 Sqoop2: job ID column is too small
    
    Increased it to 50

:100644 100644 bb01f03... 200293d... M  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoConstants.java
:100644 100644 f6bd5b5... 331780b... M  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
:100644 100644 0a196f0... b86d5f8... M  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java
Diffs (updated)
-----

  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoConstants.java bb01f0308499145e20ebebae1473591ba17227d0
  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java f6bd5b5c560486f85db0676ae74544a8c8420b51
  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java 0a196f03a39fd0c802de643988b73fc804ec68e1

Diff: https://reviews.apache.org/r/14351/diff/
Testing
-------

Ran sqoop tests and verified fix locally.
Thanks,

Abraham Elmahrek

+
Abraham Elmahrek 2013-10-01, 20:45
+
Jarek Cecho 2013-10-02, 01:08
+
Abraham Elmahrek 2013-10-02, 01:35