sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gwen Shapira" <gshap...@cloudera.com>
Subject Re: Review Request 27085: SQOOP-1589: Sqoop2: Create common constants, error codes, and queries
Date Mon, 03 Nov 2014 19:35:45 GMT


> On Oct. 31, 2014, 5:28 p.m., Gwen Shapira wrote:
> > repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryInsertUpdateDeleteSelectQuery.java,
line 22
> > <https://reviews.apache.org/r/27085/diff/3/?file=735743#file735743line22>
> >
> >     Perhaps we want to separate this farther into DML (insert, update, delete) and
queries (that modify nothing)?
> >     I'm not sure if it will make things more or less readable, would like to hear
your thoughts on this.
> 
> Abraham Elmahrek wrote:
>     Hmmm... perhaps. Follow up Jira?

Sure. and not high priority.


> On Oct. 31, 2014, 5:28 p.m., Gwen Shapira wrote:
> > repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoError.java,
line 25
> > <https://reviews.apache.org/r/27085/diff/3/?file=735746#file735746line25>
> >
> >     I'm wondering if DB specific error messages should contain text that indicate
the DB.
> 
> Abraham Elmahrek wrote:
>     Is DERBYREPO_XXXX good enough?

Note sure. I want it to be clear in the error message so it will be easy for us, customers
and support to see what caused the error. Does using DERBYREP_XXX modify the error message
in the log? or just the stack?


- Gwen


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


On Nov. 3, 2014, 7:29 p.m., Abraham Elmahrek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27085/
> -----------------------------------------------------------
> 
> (Updated Nov. 3, 2014, 7:29 p.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1589
>     https://issues.apache.org/jira/browse/SQOOP-1589
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> commit 07f1441c6bb1a99f8e2ef81c89d2b4752fc5d76d
> 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 f25a29f... 29de641... M  pom.xml
> :100644 100644 e3345c4... 8c95c0e... M  repository/pom.xml
> :000000 100644 0000000... 37378c6... A  repository/repository-common/pom.xml
> :000000 100644 0000000... 6d2c926... A  repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryError.java
> :000000 100644 0000000... 3198b0f... A  repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java
> :000000 100644 0000000... cf3df00... A  repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryInsertUpdateDeleteSelectQuery.java
> :000000 100644 0000000... 173dcb8... A  repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositorySchemaConstants.java
> :100644 100644 6ad6d64... 9be96db... M  repository/repository-derby/pom.xml
> :100644 100644 aad219e... 769544b... M  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoError.java
> :100644 100644 7f19c28... 726d4ab... M  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
> :100644 100644 02b11fc... c159183... M  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaInsertUpdateDeleteSelectQuery.java
> :100644 100644 85140d5... 9a64ad9... M  repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestJobHandling.java
> :100644 100644 37343d3... b96b2d7... M  repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestLinkHandling.java
> :100644 100644 67baaa5... 090a0b1... M  server/pom.xml
> :100644 100644 7a80710... 954fc68... M  test/pom.xml
> 
> 
> Diffs
> -----
> 
>   pom.xml 3449a6f 
>   repository/pom.xml e3345c4 
>   repository/repository-common/pom.xml PRE-CREATION 
>   repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepoConstants.java
PRE-CREATION 
>   repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryError.java
PRE-CREATION 
>   repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java
PRE-CREATION 
>   repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryInsertUpdateDeleteSelectQuery.java
PRE-CREATION 
>   repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositorySchemaConstants.java
PRE-CREATION 
>   repository/repository-derby/pom.xml 6ad6d64 
>   repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoConstants.java
8fbf47f 
>   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
b324f4f 
>   repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaInsertUpdateDeleteSelectQuery.java
c894d06 
>   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
dabb08b 
>   server/pom.xml be2ee9b 
>   test/pom.xml 956aeb7 
> 
> 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