Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Sqoop, mail # dev - Review Request: SQOOP-943 Sqoop2: Repository wide configuration of immutable disk structures


Copy link to this message
-
Review Request: SQOOP-943 Sqoop2: Repository wide configuration of immutable disk structures
Jarek Cecho 2013-03-11, 04:53

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

Review request for Sqoop.
Description
-------

I've promote the idea of immutable on disk structures to a first class citizen. I've also removed similar concept from JDBC Repository implementation as it would be duplicating functionality.
This addresses bug SQOOP-943.
    https://issues.apache.org/jira/browse/SQOOP-943
Diffs
-----

  core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java 32df1e5ddb500c8447ec4d567af73181865e2da8
  core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryContext.java 8989fb6fb00a8278f7d6d457dbe5b9193fd2d58e
  core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryProvider.java 3339c597b92131ed87b3ee1a95c1c7d75c5212dc
  core/src/main/java/org/apache/sqoop/repository/RepoConfigurationConstants.java 8939f8a12356155a5bb76b4d8da40ae82abc1506
  core/src/main/java/org/apache/sqoop/repository/Repository.java d6ec3037c9460cd0572fa2d547781297e4ff9341
  core/src/main/java/org/apache/sqoop/repository/RepositoryError.java 4cae7bafe4086b5c5816b57c5b1eee523f6ece7c
  core/src/main/java/org/apache/sqoop/repository/RepositoryManager.java 955306d1536a53d5ec93b17c2ce1255f3f06df54
  dist/src/main/server/conf/sqoop.properties 5131aad897cb765e84d3aec32e0eb05846f01676
  test/src/main/java/org/apache/sqoop/test/minicluster/SqoopMiniCluster.java 3620c28a6323f37302ad6a3d877163cf09d9e20f

Diff: https://reviews.apache.org/r/9849/diff/
Testing
-------

Unit and integration tests seems to be passing.
Thanks,

Jarek Cecho