You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kl...@apache.org on 2016/06/16 17:23:41 UTC
[21/23] incubator-geode git commit: GEODE-1543: Change the log level
from error to info
GEODE-1543: Change the log level from error to info
* When multiple peers join the cluster conncurrently, only the first one that joins will distribute the index creation and other will try to create the index locally because of the distributed message and then log an error afterwards.
* This is an expected behaviour and hence should not be logged as error but rather as a info.
This closes #156
Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/612a9d99
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/612a9d99
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/612a9d99
Branch: refs/heads/feature/GEODE-835
Commit: 612a9d99566ac71421e5cd986c6ce6e391dedd30
Parents: f59e8d0
Author: nabarun <nn...@pivotal.io>
Authored: Wed Jun 15 15:35:30 2016 -0700
Committer: nabarun <nn...@pivotal.io>
Committed: Wed Jun 15 15:46:59 2016 -0700
----------------------------------------------------------------------
.../java/com/gemstone/gemfire/internal/cache/LocalRegion.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/612a9d99/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java
index 8b9664f..e5897cc 100644
--- a/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java
+++ b/geode-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java
@@ -2452,8 +2452,9 @@ public class LocalRegion extends AbstractRegion
indexes.add(qs.createIndex(icd.getIndexName(), icd.getIndexType(), icd.getIndexExpression(), fromClause, icd.getIndexImportString(), !isOverflowToDisk));
}
- } catch (Exception ex) {
- logger.error("Failed to create index {} on region {} with exception: {}", icd.getIndexName(), this.getFullPath(), ex);
+ }
+ catch (Exception ex) {
+ 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.