[geode] branch develop updated: GEODE-5064: Removed unused code in LocalRegion (#1790)

[nn...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

nnag pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new 6097e3d  GEODE-5064: Removed unused code in LocalRegion (#1790)
6097e3d is described below

commit 6097e3d74713efba5e11f70921ed41347f7f6cb6
Author: Jason Huynh <hu...@gmail.com>
AuthorDate: Fri Apr 13 10:33:25 2018 -0700

    GEODE-5064: Removed unused code in LocalRegion (#1790)
---
 .../apache/geode/internal/cache/InternalRegionArguments.java   | 10 ----------
 .../main/java/org/apache/geode/internal/cache/LocalRegion.java |  7 -------
 2 files changed, 17 deletions(-)

diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/InternalRegionArguments.java b/geode-core/src/main/java/org/apache/geode/internal/cache/InternalRegionArguments.java
index b9ddd8a..966e16f 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/InternalRegionArguments.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/InternalRegionArguments.java
@@ -64,7 +64,6 @@ public class InternalRegionArguments {
 
   private Object userAttribute = null;
   private List indexes;
-  private boolean declarativeIndexCreation;
 
   private Map<String, CacheServiceProfile> cacheServiceProfiles;
 
@@ -287,15 +286,6 @@ public class InternalRegionArguments {
     return this.indexes;
   }
 
-  public InternalRegionArguments setDeclarativeIndexCreation(boolean value) {
-    this.declarativeIndexCreation = value;
-    return this;
-  }
-
-  public boolean getDeclarativeIndexCreation() {
-    return this.declarativeIndexCreation;
-  }
-
   public InternalRegionArguments addCacheServiceProfile(CacheServiceProfile profile) {
     if (this.cacheServiceProfiles == null) {
       this.cacheServiceProfiles = new HashMap<>();
diff --git a/geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java b/geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java
index 23f75c7..3aab945 100644
--- a/geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java
+++ b/geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java
@@ -2433,13 +2433,6 @@ public class LocalRegion extends AbstractRegion implements LoaderHelperFactory,
           logger.info("Failed to create index {} on region {} with exception: {}",
               icd.getIndexName(), this.getFullPath(), ex);
 
-          // Check if the region index creation is from cache.xml, in that case throw exception.
-          // Other case is when bucket regions are created dynamically, in that case ignore the
-          // exception.
-          if (internalRegionArgs.getDeclarativeIndexCreation()) {
-            throw new InternalGemFireError(LocalizedStrings.GemFireCache_INDEX_CREATION_EXCEPTION_1
-                .toLocalizedString(icd.getIndexName(), this.getFullPath()), ex);
-          }
         }
       }
     } finally {

-- 
To stop receiving notification emails like this one, please contact
nnag@apache.org.