asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Westmann (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: fix some log levels
Date Wed, 19 Oct 2016 00:27:44 GMT
Till Westmann has submitted this change and it was merged.

Change subject: fix some log levels
......................................................................


fix some log levels

Change-Id: I4810f02e1be0b4a4b5238afa7b929ff547beef06
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1302
Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Reviewed-by: Michael Blow <mblow@apache.org>
---
M hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/lifecycle/LifeCycleComponentManager.java
1 file changed, 5 insertions(+), 5 deletions(-)

Approvals:
  Michael Blow: Looks good to me, approved
  Jenkins: Verified; No violations found; Verified



diff --git a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/lifecycle/LifeCycleComponentManager.java
b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/lifecycle/LifeCycleComponentManager.java
index d4d2bce..d6961b3 100644
--- a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/lifecycle/LifeCycleComponentManager.java
+++ b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/lifecycle/LifeCycleComponentManager.java
@@ -119,8 +119,8 @@
                     }
                     componentDumpStream = new FileOutputStream(f);
                 }
-                if (LOGGER.isLoggable(Level.SEVERE)) {
-                    LOGGER.severe("Stopping component instance" + component.getClass().getName()
+ " dump state "
+                if (LOGGER.isLoggable(Level.INFO)) {
+                    LOGGER.info("Stopping component instance " + component.getClass().getName()
+ " dump state "
                             + dumpState + " dump path " + componentDumpPath);
                 }
                 component.stop(dumpState, componentDumpStream);
@@ -143,12 +143,12 @@
         dumpPath = configuration.get(Config.DUMP_PATH_KEY);
         if (dumpPath == null) {
             dumpPath = System.getProperty("user.dir");
-            if (LOGGER.isLoggable(Level.SEVERE)) {
-                LOGGER.severe("dump path not configured. Using current directory " + dumpPath);
+            if (LOGGER.isLoggable(Level.WARNING)) {
+                LOGGER.warning("dump path not configured. Using current directory " + dumpPath);
             }
         }
         if (LOGGER.isLoggable(Level.INFO)) {
-            LOGGER.severe("LifecycleComponentManager configured " + this);
+            LOGGER.info("LifecycleComponentManager configured " + this);
         }
         configured = true;
     }

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1302
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4810f02e1be0b4a4b5238afa7b929ff547beef06
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <tillw@apache.org>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mblow@apache.org>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>

Mime
View raw message