You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by js...@apache.org on 2017/05/08 21:38:22 UTC

geode git commit: GEODE-2876: Add logging to diagnose CI failure

Repository: geode
Updated Branches:
  refs/heads/develop 5a1b06210 -> 65821d1ea


GEODE-2876: Add logging to diagnose CI failure


Project: http://git-wip-us.apache.org/repos/asf/geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/geode/commit/65821d1e
Tree: http://git-wip-us.apache.org/repos/asf/geode/tree/65821d1e
Diff: http://git-wip-us.apache.org/repos/asf/geode/diff/65821d1e

Branch: refs/heads/develop
Commit: 65821d1ea3e19743e854b25cba408612db72d06d
Parents: 5a1b062
Author: Jared Stewart <js...@pivotal.io>
Authored: Mon May 8 14:35:57 2017 -0700
Committer: Jared Stewart <js...@pivotal.io>
Committed: Mon May 8 14:37:35 2017 -0700

----------------------------------------------------------------------
 .../org/apache/geode/management/internal/cli/GfshParser.java     | 1 +
 .../geode/management/internal/cli/remote/CommandProcessor.java   | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/geode/blob/65821d1e/geode-core/src/main/java/org/apache/geode/management/internal/cli/GfshParser.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/GfshParser.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/GfshParser.java
index 20ae022..a403469 100755
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/GfshParser.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/GfshParser.java
@@ -179,6 +179,7 @@ public class GfshParser extends SimpleParser {
   @Override
   public GfshParseResult parse(String userInput) {
     String rawInput = convertToSimpleParserInput(userInput);
+    logWrapper.info("Sending raw input [" + rawInput + "] to SimpleParser");
 
     // User SimpleParser to parse the input
     ParseResult result = super.parse(rawInput);

http://git-wip-us.apache.org/repos/asf/geode/blob/65821d1e/geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/CommandProcessor.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/CommandProcessor.java b/geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/CommandProcessor.java
index c2c6e14..caec989 100755
--- a/geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/CommandProcessor.java
+++ b/geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/CommandProcessor.java
@@ -121,8 +121,8 @@ public class CommandProcessor {
         }
       } catch (CommandProcessingException e) { // expected from Parser
         setLastExecutionStatus(1);
-        if (logWrapper.infoEnabled()) {
-          logWrapper.info("Could not parse \"" + cmdStmt.getCommandString() + "\".", e);
+        if (logWrapper.warningEnabled()) {
+          logWrapper.warning("Could not parse \"" + cmdStmt.getCommandString() + "\".", e);
         }
         return ResultBuilder.createParsingErrorResult(e.getMessage());
       } catch (NotAuthorizedException e) {