Skip to content

Conversation

andyyang890
Copy link
Collaborator

Backport 1/1 commits from #154802.

/cc @cockroachdb/release


This patch fixes a bug where the memory monitor wouldn't reclaim the
memory allocated to events corresponding to unwatched column families
for a changefeed that targets only a subset of a table's families.

Fixes #154776

Release note (bug fix): A bug where a changefeed targeting only a subset
of a table's column families could become stuck has been fixed.


Release justification: fix for high-severity bug

@andyyang890 andyyang890 requested a review from asg0451 October 8, 2025 00:21
@andyyang890 andyyang890 requested a review from a team as a code owner October 8, 2025 00:21
Copy link

blathers-crl bot commented Oct 8, 2025

Thanks for opening a backport.

Before merging, please confirm that it falls into one of the following categories (select one):

  • Non-production code changes. Includes test-only changes, build system changes, etc.
  • Fixes for serious issues. Defined in the policy as correctness, stability, or security issues, data corruption/loss, significant performance regressions, breaking working and widely used functionality, or an inability to detect and debug production issues.
  • Other approved changes. These changes must be gated behind a disabled-by-default feature flag unless there is a strong justification not to.

Add a brief release justification to the PR description explaining your selection.

Also, confirm that the change does not break backward compatibility and complies with all aspects of the backport policy.

All backports must be reviewed by the TL and EM for the owning area.

@blathers-crl blathers-crl bot added backport Label PR's that are backports to older release branches T-cdc labels Oct 8, 2025
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link

blathers-crl bot commented Oct 8, 2025

✅ PR #155024 is compliant with backport policy

Confidence: high
Critical bug criteria met: [Stability or security issues]
Backward compatible: true
Explanation: The pull request qualifies for a policy exemption due to the inclusion of a valid release justification statement 'Release justification: fix for high-severity bug', which identifies the backport as addressing a high-severity issue. The changes primarily relate to a bug fix in the 'changefeedccl' component, where a memory monitoring issue would lead to the changefeed process becoming stuck. This addresses a critical bug as per the policy since it fixes a bug that can affect the stability of the DB and disrupt its operations. Moreover, the changes are limited to the relevant CCL package and include tests, indicating a targeted and verified fix. There are no feature flags needed since this is a critical bug fix.

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

This patch fixes a bug where the memory monitor wouldn't reclaim the
memory allocated to events corresponding to unwatched column families
for a changefeed that targets only a subset of a table's families.

Release note (bug fix): A bug where a changefeed targeting only a subset
of a table's column families could become stuck has been fixed.
@andyyang890 andyyang890 merged commit 3d52589 into cockroachdb:release-24.3 Oct 8, 2025
16 checks passed
@andyyang890 andyyang890 deleted the backport24.3-154802 branch October 8, 2025 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Label PR's that are backports to older release branches T-cdc target-release-24.3.22

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants