[ 
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

Reply via email to