[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2020-02-05 Thread Alex Petrov (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17030518#comment-17030518 ] Alex Petrov commented on CASSANDRA-15429: - [~dcapwell] the [last

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2020-02-04 Thread Yifan Cai (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17030034#comment-17030034 ] Yifan Cai commented on CASSANDRA-15429: --- Thanks [~ifesdjeen]! The patch LGTM. +1 Thanks

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2020-02-04 Thread David Capwell (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17030030#comment-17030030 ] David Capwell commented on CASSANDRA-15429: --- The patch looks fine to me, but I want a

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2020-02-04 Thread David Capwell (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17030029#comment-17030029 ] David Capwell commented on CASSANDRA-15429: --- The linked Circle run is

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2020-02-04 Thread David Capwell (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17030027#comment-17030027 ] David Capwell commented on CASSANDRA-15429: --- Based off the patch I did the following greps

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2020-02-04 Thread David Capwell (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17030013#comment-17030013 ] David Capwell commented on CASSANDRA-15429: --- well, as long as help is working... =D Ill

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2019-12-13 Thread Yifan Cai (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16995892#comment-16995892 ] Yifan Cai commented on CASSANDRA-15429: --- [~djoshi], thanks! Changes are made to address your

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2019-12-12 Thread Dinesh Joshi (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16995177#comment-16995177 ] Dinesh Joshi commented on CASSANDRA-15429: -- Hi [~yifanc], thanks for the PR. I have left a

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2019-12-10 Thread David Capwell (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16993122#comment-16993122 ] David Capwell commented on CASSANDRA-15429: --- LGTM +1 > Support NodeTool for in-jvm dtest

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2019-12-10 Thread Yifan Cai (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16993080#comment-16993080 ] Yifan Cai commented on CASSANDRA-15429: --- Thanks [~dcapwell]! PRs have been updated to address

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2019-12-10 Thread David Capwell (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16993049#comment-16993049 ] David Capwell commented on CASSANDRA-15429: --- Did another pass. Mostly LGTM; left a few

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2019-12-10 Thread David Capwell (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16992882#comment-16992882 ] David Capwell commented on CASSANDRA-15429: --- We should refactor NodeProbe and

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2019-12-10 Thread Yifan Cai (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16992778#comment-16992778 ] Yifan Cai commented on CASSANDRA-15429: --- Thanks [~ifesdjeen] {quote}Can we hide nodetool

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2019-12-10 Thread Alex Petrov (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16992286#comment-16992286 ] Alex Petrov commented on CASSANDRA-15429: - Not an official review, just glanced over the

[jira] [Commented] (CASSANDRA-15429) Support NodeTool for in-jvm dtest

2019-11-18 Thread Yifan Cai (Jira)
[ https://issues.apache.org/jira/browse/CASSANDRA-15429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16976807#comment-16976807 ] Yifan Cai commented on CASSANDRA-15429: --- [~drohrer] and I co-worked on it. The changes in the