[
https://issues.apache.org/jira/browse/CASSANDRA-20081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18052746#comment-18052746
]
guo Maxwell commented on CASSANDRA-20081:
-----------------------------------------
Hi, [~arvindk12] [~Jyothsnakonisa], I have already looked at the first round of
code.
I have a question here: why you add a new column compaction_type ? What are
the benefits?
see the
[CASSANDRA-18061|https://issues.apache.org/jira/browse/CASSANDRA-18061], we
just wanted to keep all the compaction information in the
column compaction_properties, and that is the reason we used a map here.
The compaction type has been stored in compaction_properties for 3 years. I am
not sure if the information in this column has been used by users. If it has,
how can we ensure forward compatibility for our behavior changes?
> Enhance nodetool compactionhistory to report compaction type and strategy
> -------------------------------------------------------------------------
>
> Key: CASSANDRA-20081
> URL: https://issues.apache.org/jira/browse/CASSANDRA-20081
> Project: Apache Cassandra
> Issue Type: New Feature
> Components: Tool/nodetool
> Reporter: Brad Schoening
> Assignee: Arvind Kandpal
> Priority: Normal
> Time Spent: 1h 10m
> Remaining Estimate: 0h
>
> It would be useful for compactionhistory to have:
> a) updated documentation for nodetool [compactionhistory
> |https://cassandra.apache.org/doc/5.0/cassandra/managing/tools/nodetool/compactionhistory.html]
> describing the columns *rows_merged* and {*}compaction_properties{*}. This
> StackOverflow article provides a good descriptions of rows_merged: [What
> does rows_merged mean in
> compactionhistory|https://stackoverflow.com/questions/27566558/what-does-rows-merged-mean-in-compactionhistory]
> b) the compaction strategy used: STCS, LCS, TWCS, UCS and sub-properties. For
> example, with level, the from-level and to-level could be reported.
> c) also useful would be to make the bytes in and out human readable by
> default, since there is already an -f option for machine readable output.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
