> On May 4, 2017, 5:29 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/CommandProcessor.java > > Line 124 (original), 125 (patched) > > <https://reviews.apache.org/r/58996/diff/1/?file=1708729#file1708729line125> > > > > we are not showing info level logging in our current logs? > > > > Also the best place to see what's being actually parsed is in > > GfshParser.parse method. That's where we are handing the input string to > > SpringShell's parsing method.
I think this really does belong at "warning" level. (If a user tries to execute a command that can't be parsed, I think that should still be logged even if they have set the log-level to WARN.) That said, I'm a bit puzzled by the current behavior. The log statement does show up in the log, but it appears to be missing the accompanying stack trace. ``` [vm3] Command result for <deploy --jar=/tmp/junit3108566533782045832/jar1.jar>: [vm3] Could not parse command string. deploy --jar=/tmp/junit3108566533782045832/jar1.jar ``` The other change should fix this by passing the full stacktrace back to the client in the gfsh ErrorResult. (I think this would also make it easier for a real user either to diagnose their failure or to file a descriptive bug report.) - Jared ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58996/#review173930 ----------------------------------------------------------- On May 4, 2017, 5:12 p.m., Jared Stewart wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58996/ > ----------------------------------------------------------- > > (Updated May 4, 2017, 5:12 p.m.) > > > Review request for geode. > > > Repository: geode > > > Description > ------- > > GEODE-2876: Add logging to diagnose CI failure > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/CommandProcessor.java > c2c6e1425d71af9d2ea59046b17afd70ad30dd68 > > > Diff: https://reviews.apache.org/r/58996/diff/1/ > > > Testing > ------- > > > Thanks, > > Jared Stewart > >
