----------------------------------------------------------- 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 > >
