Review Board 1.7.22


SQOOP-1235 Sqoop2: Repository should be immutable by default

Review Request #15637 - Created Nov. 18, 2013 and updated

Jarek Cecho
SQOOP-1235
Reviewers
Sqoop
sqoop-sqoop2
Changed the default value to true.
Unit and integration tests seems to be passing.
core/src/main/java/org/apache/sqoop/repository/RepositoryManager.java
Revision 25d5d9d03532c677b524512aa0d175c969805b10 New Change
[20] 110 lines
[+20] [+] public synchronized void initialize() {
111
          repoProviderClassName, ex);
111
          repoProviderClassName, ex);
112
    }
112
    }
113

    
   
113

   
114
    provider.initialize(context);
114
    provider.initialize(context);
115

    
   
115

   
116
    if(!context.getBoolean(RepoConfigurationConstants
116
    if(!context.getBoolean(RepoConfigurationConstants.SYSCFG_REPO_SCHEMA_IMMUTABLE, true)) {
117
      .SYSCFG_REPO_SCHEMA_IMMUTABLE, false)) {

   
118
      LOG.info("Creating or upgrading on disk structures if necessary");
117
      LOG.info("Creating or upgrading on disk structures if necessary");
119
      provider.getRepository().createOrUpdateInternals();
118
      provider.getRepository().createOrUpdateInternals();
120
    }
119
    }
121

    
   
120

   
122
    if(!provider.getRepository().haveSuitableInternals()) {
121
    if(!provider.getRepository().haveSuitableInternals()) {
[+20] [20] 45 lines
  1. core/src/main/java/org/apache/sqoop/repository/RepositoryManager.java: Loading...