You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by bs...@apache.org on 2015/11/05 21:35:50 UTC
[13/50] [abbrv] incubator-geode git commit: GEODE-469: Don't log a
warning if a region is destroyed during creation
GEODE-469: Don't log a warning if a region is destroyed during creation
This test saw a suspect string because we logged a warning about a
RegionDestroyedException during region creation. I don't think this
warning should be logged, we don't log warnings for CancelExceptions.
Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/320b6e4b
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/320b6e4b
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/320b6e4b
Branch: refs/heads/feature/GEODE-77
Commit: 320b6e4b6084791358e7f13ebb4ef08b7b0fb9ec
Parents: fa3a47b
Author: Dan Smith <up...@apache.org>
Authored: Tue Oct 27 17:30:11 2015 -0700
Committer: Dan Smith <up...@apache.org>
Committed: Tue Oct 27 17:34:22 2015 -0700
----------------------------------------------------------------------
.../java/com/gemstone/gemfire/internal/cache/LocalRegion.java | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/320b6e4b/gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java
----------------------------------------------------------------------
diff --git a/gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java b/gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java
index b5e6171..a3b8d35 100644
--- a/gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java
+++ b/gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java
@@ -1086,12 +1086,9 @@ public class LocalRegion extends AbstractRegion
}
}
success = true;
- } catch (CancelException e) {
+ } catch (CancelException | RegionDestroyedException | RedundancyAlreadyMetException e) {
// don't print a call stack
throw e;
- } catch(RedundancyAlreadyMetException e) {
- //don't log this
- throw e;
} catch (final RuntimeException validationException) {
logger.warn(LocalizedMessage.create(LocalizedStrings.LocalRegion_INITIALIZATION_FAILED_FOR_REGION_0,
getFullPath()), validationException);