Skip to content

Dont validate CompactionInterval when compaction is disabled#3664

Merged
zymap merged 1 commit into
apache:masterfrom
gaozhangmin:compaction-config
Nov 24, 2022
Merged

Dont validate CompactionInterval when compaction is disabled#3664
zymap merged 1 commit into
apache:masterfrom
gaozhangmin:compaction-config

Conversation

@gaozhangmin
Copy link
Copy Markdown
Contributor

@gaozhangmin gaozhangmin commented Nov 22, 2022

Motivation

Currently, We cannot disable minorCompaction and majorCompaction by setting interval to -1, with gcWaitTime is enabled.

Changes

Don't validate CompactionInterval if compaction is disabled

Copy link
Copy Markdown
Member

@horizonzy horizonzy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zymap zymap merged commit 948d94f into apache:master Nov 24, 2022
@hangc0276 hangc0276 added this to the 4.16.0 milestone Nov 25, 2022
yaalsn pushed a commit to yaalsn/bookkeeper that referenced this pull request Jan 30, 2023
…3664)

### Motivation
Currently, We cannot disable minorCompaction and majorCompaction by setting interval to -1, with `gcWaitTime` is  enabled.

### Changes
Don't validate CompactionInterval if compaction is disabled
Ghatage pushed a commit to sijie/bookkeeper that referenced this pull request Jul 12, 2024
…3664)

### Motivation
Currently, We cannot disable minorCompaction and majorCompaction by setting interval to -1, with `gcWaitTime` is  enabled.

### Changes
Don't validate CompactionInterval if compaction is disabled
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants