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

Adjust per_changefeed_limit to 128MiB #84686

Merged
merged 1 commit into from
Jul 20, 2022

Conversation

shermanCRL
Copy link
Contributor

@shermanCRL shermanCRL commented Jul 20, 2022

Adjust changefeed.memory.per_changefeed_limit to 128MiB. Default of 1GiB could pressure GC, causing GC assist, affecting foreground traffic.

Informs #84582

Release Notes (enterprise change): Reduce foreground latency impact when performing changefeed backfills.

Default of 1GiB could pressure GC, affecting foreground traffic
@shermanCRL shermanCRL requested a review from a team as a code owner July 20, 2022 00:08
@shermanCRL shermanCRL requested review from gh-casper and removed request for a team July 20, 2022 00:08
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@shermanCRL shermanCRL requested review from miretskiy and ajwerner and removed request for gh-casper July 20, 2022 00:18
@shermanCRL
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Jul 20, 2022

Build succeeded:

@shermanCRL
Copy link
Contributor Author

blathers backport 21.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants