cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hiroyuki Nishi (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-12035) Structure for tpstats output (JSON, YAML)
Date Fri, 08 Jul 2016 13:50:10 GMT

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

Hiroyuki Nishi commented on CASSANDRA-12035:
--------------------------------------------

Hi [~ifesdjeen],

Thank you for your review and feedback!

I think so. I tried refactor DefaultPrinter on TpStatsPrinter as the following.

https://github.com/yhnishi/cassandra/commit/cbdead281acf135fdcc66464ad3a7bfc1fd881d2

Is this what you mean by your suggestion?

> Structure for tpstats output (JSON, YAML)
> -----------------------------------------
>
>                 Key: CASSANDRA-12035
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-12035
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Hiroyuki Nishi
>            Assignee: Hiroyuki Nishi
>            Priority: Minor
>         Attachments: CASSANDRA-12035-trunk.patch, tablestats_result.json, tablestats_result.txt,
tablestats_result.yaml, tpstats_result.json, tpstats_result.txt, tpstats_result.yaml
>
>
> In CASSANDRA-5977, some extra output formats such as JSON and YAML were added for nodetool
tablestats. 
> Similarly, I would like to add the output formats in nodetool tpstats.
> Also, I tried to refactor the tablestats's code about the output formats to integrate
the existing code with my code.
> Please review the attached patch.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message