sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Veena Basavaraj" <vbasava...@cloudera.com>
Subject Re: Review Request 26941: SQOOP-1557: SQ_CONFIGURABLE ( for entities who own configs)
Date Tue, 21 Oct 2014 18:26:01 GMT

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

(Updated Oct. 21, 2014, 11:25 a.m.)


Review request for Sqoop.


Repository: sqoop-sqoop2


Description
-------

see JIRA for details

there is whitespace, that will be addressed once the reviews for the functionality


Diffs (updated)
-----

  common/src/main/java/org/apache/sqoop/model/MConfigurableType.java PRE-CREATION 
  common/src/main/java/org/apache/sqoop/model/MConnector.java 2f42191 
  common/src/main/java/org/apache/sqoop/model/MDriver.java 685439e 
  core/src/main/java/org/apache/sqoop/driver/Driver.java f1b45bb 
  core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java f06fd0c 
  core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryHandler.java 5a8e026 
  core/src/main/java/org/apache/sqoop/repository/Repository.java 74a9e12 
  core/src/test/java/org/apache/sqoop/repository/TestJdbcRepository.java 34bd8a5 
  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoConstants.java
40dcc49 
  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepoError.java
3e4a4a9 
  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
c888910 
  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaConstants.java
59773e1 
  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaQuery.java
44ec2e3 
  repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/DerbyTestCase.java
366e4ee 
  repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestConnectorHandling.java
68a173b 
  repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestDriverHandling.java
95fbe07 
  repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestJobHandling.java
a15bda9 
  tools/src/main/java/org/apache/sqoop/tools/tool/RepositoryLoadTool.java 76ebd3b 

Diff: https://reviews.apache.org/r/26941/diff/


Testing
-------

yes


Thanks,

Veena Basavaraj


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