sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Abraham Elmahrek" <...@cloudera.com>
Subject Re: Review Request 27085: SQOOP-1589: Sqoop2: Create common constants, error codes, and queries
Date Fri, 24 Oct 2014 19:34:49 GMT


> On Oct. 24, 2014, 5:15 a.m., Qian Xu wrote:
> >

Qian, the comments you've left are oriented around code that I haven't written or modified.
Let's have follow up Jiras to address the issues you've brought up?


- Abraham


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


On Oct. 23, 2014, 5:13 p.m., Abraham Elmahrek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27085/
> -----------------------------------------------------------
> 
> (Updated Oct. 23, 2014, 5:13 p.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-158
>     https://issues.apache.org/jira/browse/SQOOP-158
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> commit 370f476b1834c390993b31007b0fc7e1a184b891
> Author: Abraham Elmahrek <abraham@elmahrek.com>
> Date:   Wed Oct 22 18:30:28 2014 -0700
> 
>     SQOOP-1589: Sqoop2: Create common constants, error codes, and queries
> 
> :100644 100644 7d78826... e41cd6c... M  core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryHandler.java
> :000000 100644 0000000... a7ebd50... A  core/src/main/java/org/apache/sqoop/repository/JdbcRepositorySchemaConstants.java
> :000000 100644 0000000... eb05567... A  core/src/main/java/org/apache/sqoop/repository/JdbcRepositorySchemaQuery.java
> :100644 100644 f684e85... 097a185... M  core/src/main/java/org/apache/sqoop/repository/RepositoryError.java
> :100644 100644 aad219e... 769544b... M  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoError.java
> :100644 100644 633e9df... 6764b14... M  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
> :100644 100644 ce1830e... ff8962c... M  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java
> :100644 100644 85140d5... aab1e95... M  repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestJobHandling.java
> :100644 100644 37343d3... 9bcf3c8... M  repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestLinkHandling.java
> 
> 
> Diffs
> -----
> 
>   core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryHandler.java 7d78826 
>   core/src/main/java/org/apache/sqoop/repository/JdbcRepositorySchemaConstants.java PRE-CREATION

>   core/src/main/java/org/apache/sqoop/repository/JdbcRepositorySchemaQuery.java PRE-CREATION

>   core/src/main/java/org/apache/sqoop/repository/RepositoryError.java f684e85 
>   repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoError.java
aad219e 
>   repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
633e9df 
>   repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java
ce1830e 
>   repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestJobHandling.java
85140d5 
>   repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestLinkHandling.java
37343d3 
> 
> Diff: https://reviews.apache.org/r/27085/diff/
> 
> 
> Testing
> -------
> 
> mvn clean verify + start sqoop server without issues.
> 
> 
> Thanks,
> 
> Abraham Elmahrek
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message