cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Petrov (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-12035) Structure for tpstats output (JSON, YAML)
Date Wed, 13 Jul 2016 09:58:20 GMT


Alex Petrov commented on CASSANDRA-12035:

So changes, summarised: 

  * {{TableStats}} and {{TpStats}} both provide {{json}} and {{yaml}} formatters now. 
  * {{StatsHolder}} is now an interface implemented by {{TpStatsHolder}} and {{TableStatsHolder}}
  * {{StatsPrinter}} now holds default implementations for {{json}} and {{yaml}} printers
  * {{TableStats}} logic is moved to the corresponding holder 

+1 from my side as well.


> Structure for tpstats output (JSON, YAML)
> -----------------------------------------
>                 Key: CASSANDRA-12035
>                 URL:
>             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_output.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
> 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

View raw message