[ https://issues.apache.org/jira/browse/CASSANDRA-16057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17193755#comment-17193755 ]
Yifan Cai commented on CASSANDRA-16057: --------------------------------------- Adding the PR to trunk: https://github.com/apache/cassandra/pull/749 (The CI won't run until the above PR to API is merged and a new release is out) The PR did mainly 2 things. - Made the {{out/err}} stream in the nodetool and commands pluggable. {{ConsoleOutputProvider}} is introduced to contain both stream. Bunch of files are touch because of it. - Added {{CapturingConsoleOutputProvider}} to save the outputs and pass them to {{NodeToolResult}}. {{NodeToolTest.java}} shows the usage of asserting with the captured logs in nodetool result. > Should update in-jvm dtest to expose stdout and stderr for nodetool > ------------------------------------------------------------------- > > Key: CASSANDRA-16057 > URL: https://issues.apache.org/jira/browse/CASSANDRA-16057 > Project: Cassandra > Issue Type: Improvement > Components: Test/dtest/java > Reporter: David Capwell > Assignee: Yifan Cai > Priority: Normal > > Many nodetool commands output to stdout or stderr so running nodetool using > in-jvm dtest should expose that to tests. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org