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
Copy link to this message
-
Re: Review Request 14351: SQOOP-1208 Sqoop2: job ID column is too small
Jarek Cecho 2013-09-26, 21:13

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14351/#review26421
-----------------------------------------------------------
Hi Abe,
thank you very much for taking this one up! I believe that we also needs to cover upgrade path when one is updating derby repository from prior Sqoop versions. Would you mind adding that?

Jarcec

- Jarek Cecho
On Sept. 26, 2013, 8:02 p.m., Abraham Elmahrek wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14351/
> -----------------------------------------------------------
>
> (Updated Sept. 26, 2013, 8:02 p.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 2ee24e0ca20925523cb3c8a87d9c414bcdd80293
> 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 0a196f0... 4184129... M repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java
>
>
> Diffs
> -----
>
>   repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java 0a196f0
>
> Diff: https://reviews.apache.org/r/14351/diff/
>
>
> Testing
> -------
>
> Ran sqoop tests and verified fix locally.
>
>
> Thanks,
>
> Abraham Elmahrek
>
>

+
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
+
Abraham Elmahrek 2013-10-01, 07:00
+
Abraham Elmahrek 2013-10-01, 20:45
+
Jarek Cecho 2013-10-02, 01:08
+
Abraham Elmahrek 2013-10-02, 01:35