[ 
https://issues.apache.org/jira/browse/CASSANDRA-16057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17198489#comment-17198489
 ] 

Yifan Cai commented on CASSANDRA-16057:
---------------------------------------

Thanks for all the feedbacks. 

[~stefan.miklosovic], as David mentioned, this patch does what you described. 
Capture the nodetool output per instance. All nodetools are updated to write to 
a designated output. Additionally, it captures *only* the nodetool output. 
Meanwhile, redirecting the system out/err captures the server logs, which could 
make assertion a bit more complicated. 

[~ifesdjeen][~dcapwell], I have added 1 commit to address the comments. 
* rename class and variable
* use public final fields instead of methods to avoid resource warnings

> 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
>          Time Spent: 2h 20m
>  Remaining Estimate: 0h
>
> 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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to