Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the compaction phase one loop timeout configurable #11206

Conversation

codelipenghui
Copy link
Contributor

No description provided.

@codelipenghui codelipenghui requested review from lhotari, sijie, eolivelli, 315157973, addisonj, hangc0276 and merlimat and removed request for lhotari July 3, 2021 02:31
@codelipenghui codelipenghui self-assigned this Jul 3, 2021
@codelipenghui codelipenghui added this to the 2.9.0 milestone Jul 3, 2021
@codelipenghui codelipenghui added release/2.7.3 release/2.8.1 type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages labels Jul 3, 2021
conf/broker.conf Outdated
@@ -450,6 +450,10 @@ brokerServiceCompactionMonitorIntervalInSeconds=60
# Using a value of 0, is disabling compression check.
brokerServiceCompactionThresholdInBytes=0

# Timeout for the compaction phase one loop.
# If the execution time of the compaction phase one loop exceeds this time, the compaction will not proceed.
brokerServiceCompactionPhaseOneLoopTimeInSeconds=10
Copy link
Contributor

Choose a reason for hiding this comment

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

Most of the other timeouts are set to 30seconds. Maybe we could also switch this default value to 30.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Make sense.

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

Lgtm

@codelipenghui codelipenghui merged commit 96e4fd6 into apache:master Jul 4, 2021
@codelipenghui codelipenghui deleted the penghui/config-compaction-phase-one-timeout branch July 4, 2021 00:48
codelipenghui added a commit that referenced this pull request Jul 4, 2021
@codelipenghui codelipenghui added the cherry-picked/branch-2.7 Archived: 2.7 is end of life label Jul 4, 2021
codelipenghui added a commit that referenced this pull request Jul 7, 2021
@codelipenghui codelipenghui added the cherry-picked/branch-2.8 Archived: 2.8 is end of life label Jul 7, 2021
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
@codelipenghui codelipenghui restored the penghui/config-compaction-phase-one-timeout branch May 17, 2022 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-picked/branch-2.7 Archived: 2.7 is end of life cherry-picked/branch-2.8 Archived: 2.8 is end of life release/2.7.3 release/2.8.1 type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants