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

storage: expose new compaction concurrency env var #113052

Merged
merged 1 commit into from Oct 30, 2023

Conversation

jbowens
Copy link
Collaborator

@jbowens jbowens commented Oct 25, 2023

Add a new COCKROACH_COMPACTION_CONCURRENCY environment variable to control the
maximum number of concurrent compactions that a single store will schedule.
This environment variable will supersede the old COCKROACH_ROCKSDB_CONCURRENCY
environment variable which was undocumented and was unfortunately named.

Epic: none
Release note (ops change): Introduced a new documented environment variable
that allows an operator to configure the compaction concurrency.

@blathers-crl
Copy link

blathers-crl bot commented Oct 25, 2023

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@jbowens jbowens force-pushed the concurrency-env branch 2 times, most recently from 11cffb1 to e11d02d Compare October 25, 2023 18:20
Add a new COCKROACH_COMPACTION_CONCURRENCY environment variable to control the
maximum number of concurrent compactions that a single store will schedule.
This environment variable will supersede the old COCKROACH_ROCKSDB_CONCURRENCY
environment variable which was undocumented and was unfortunately named.

Epic: none
Release note (ops change): Introduced a new documented environment variable
that allows an operator to configure the compaction concurrency.
@jbowens jbowens marked this pull request as ready for review October 25, 2023 19:13
@jbowens jbowens requested a review from a team as a code owner October 25, 2023 19:13
@nicktrav
Copy link
Collaborator

Nice.

Copy link
Member

@itsbilal itsbilal left a comment

Choose a reason for hiding this comment

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

LGTM

@jbowens jbowens added the backport-23.2.x Flags PRs that need to be backported to 23.2. label Oct 30, 2023
@jbowens
Copy link
Collaborator Author

jbowens commented Oct 30, 2023

Adding backport-23.2 because we talked about doing this for the 23.2 release. However, this would constitute a feature change and requires an okay from above.

@jbowens
Copy link
Collaborator Author

jbowens commented Oct 30, 2023

TFTR!

bors r=itsbilal

@craig
Copy link
Contributor

craig bot commented Oct 30, 2023

Build succeeded:

@craig craig bot merged commit 047f45e into cockroachdb:master Oct 30, 2023
8 checks passed
@jbowens jbowens deleted the concurrency-env branch October 30, 2023 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-23.2.x Flags PRs that need to be backported to 23.2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants