[
https://issues.apache.org/jira/browse/CASSANDRA-16310?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yifan Cai updated CASSANDRA-16310:
----------------------------------
Reviewers: Yifan Cai
Status: Review In Progress (was: Patch Available)
# Having setters for {{max_top_size_partition_count}} and
{{max_top_tombstone_partition_count}} suggests that we want to tweak it at
runtime. However, the property consumers, {{TopPartitionTracker}} and
{{ColumnFamilyStore}} do not update the value once initialized. So it has no
effects when operator update the values.
# {{TopPartitionsTest}} only has the test cases for full repair. Can we add
tests that run with preview repair? (I simply replaced the option in the
nodetool commands in the tests.)
## The tests cases pass with the {{--preview}} option.
## They fail with the {{--validate}} option. No sstables are selected since
the corresponding {{PreviewKind.REPAIRED}} filters out unrepaired ones.
# The todo comment at TableStatsPrinter line#131 can be removed, since it is
implemented.
# Nit: consider adding a {{isValidation()}} method to {{PreviewKind}}. So in
{{ValidationManager#doValidation}}, we can have
{{validator.getPreviewKind().isValidation()}}, instead of
{{validator.getPreviewKind() == PreviewKind.REPAIRED}}, which is less readable.
# Document the feature.
> Track top partitions (by size and tombstone count) and display in nodetool
> tablestats
> -------------------------------------------------------------------------------------
>
> Key: CASSANDRA-16310
> URL: https://issues.apache.org/jira/browse/CASSANDRA-16310
> Project: Cassandra
> Issue Type: Improvement
> Components: Local/Other
> Reporter: Marcus Eriksson
> Assignee: Marcus Eriksson
> Priority: Normal
> Fix For: 4.x
>
>
> We should track the top partitions by size and tombstone count and display in
> {{nodetool tablestats}}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
