geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From u..@apache.org
Subject [12/17] incubator-geode git commit: GEODE-1543: Change the log level from error to info
Date Thu, 16 Jun 2016 22:07:26 GMT
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/8994be05
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/8994be05
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/8994be05

Branch: refs/heads/feature/GEODE-420
Commit: 8994be0564bb0622a4b9b99c4db10b714ebb8b76
Parents: fae0f90
Author: nabarun <nnag@pivotal.io>
Authored: Wed Jun 15 15:35:30 2016 -0700
Committer: Udo Kohlmeyer <ukohlmeyer@pivotal.io>
Committed: Fri Jun 17 08:05:58 2016 +1000

----------------------------------------------------------------------
 .../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/8994be05/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.


Mime
View raw message