[ https://issues.apache.org/jira/browse/CASSANDRA-16057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17198461#comment-17198461 ]
David Capwell commented on CASSANDRA-16057: ------------------------------------------- bq. Ultimately, formatted output is good, but structured reporting would be even better if we ever have chance to tacle this. Maybe it's a good point to continue the conversation and create a separate issue for this? +1 to separate issue. bq. We are just exposing there stdout and stderr. Please keep in mind that I need to do this against an instance, meaning I need to be able to do it such way that it will be executed in the "scope" of that node because I can spin up more clusters and I need to have "stdout" per each "jvm / classloader". Yep, this is part of the nodetool interface, which is scoped to the single instance. The implementation also deals with this by updating all nodetool commands to update an interface rather than stdout and stderr directly; this avoids concurrent node tools clobbering the capture. bq. I would love to merge 15406 very soon because it was there literally for months and it was blocked by few other tickets so I am personally little impatient here I don't mind us moving forward with 15406 without this patch; I thought you wanted to hold off given your comment in there. Because both touch the same classes, if 15406 goes first then this patch needs to update the pointer from stdout to the new interface, if we hold off 15406 then that would need to change the pointer. > 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 10m > 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: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org