Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/24618 )
Change subject: KUDU-3734 Compaction budget refinements and observability ...................................................................... Patch Set 11: (1 comment) No test in the patch exercises any new behavior. Something like: - compaction_policy-test: with an oversized candidate present, assert compaction_budget_skip_health flips to 1; then GC/shrink it (or hit the asc_max_key.empty() early return) and assert it resets to 0. - rowset_info unit check that --rowset_undo_deltas_budget_weight (e.g. 0.0, 0.5, 1.0) scales base_and_deltas_size_mb() as expected, and the clamp above holds when undo >= total. - A ts_tablet_manager / metrics test that tablets_num_compaction_budget_degraded counts only RUNNING replicas with the gauge set. What do you think? http://gerrit.cloudera.org:8080/#/c/24618/11//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/24618/11//COMMIT_MSG@84 PS11, Line 84: tablet_compaction_budget_mb Maybe this is not pressing for this particular change, but is --tablet_compaction_budget_mb a runtime flag? -- To view, visit http://gerrit.cloudera.org:8080/24618 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id63db0006c77ad1d55eb2119d99ee64499435819 Gerrit-Change-Number: 24618 Gerrit-PatchSet: 11 Gerrit-Owner: Ashwani Raina <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Martonka <[email protected]> Gerrit-Comment-Date: Tue, 22 Sep 2026 13:52:05 +0000 Gerrit-HasComments: Yes
