Skip to content

release-25.1: changefeedccl: fix TestNoBackfillAfterNonTargetColumnDrop #150286

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

Merged
merged 2 commits into from
Jul 16, 2025

Conversation

andyyang890
Copy link
Collaborator

Backport 2/2 commits from #150215.

/cc @cockroachdb/release


This patch fixes TestNoBackfillAfterNonTargetColumnDrop so that it
actually checks that no backfills occur after a non-target column drop.

Fixes #150214

Release note: None


Release justification: test-only fix

This patch replaces an incorrect usage of `require.Regexp` in
`TestNoStopAfterNonTargetColumnDrop`.

Release note: None
This patch fixes `TestNoBackfillAfterNonTargetColumnDrop` so that it
actually checks that no backfills occur after a non-target column drop.

Release note: None
@andyyang890 andyyang890 requested review from asg0451 and aerfrei July 15, 2025 21:50
@andyyang890 andyyang890 requested a review from a team as a code owner July 15, 2025 21:50
Copy link

blathers-crl bot commented Jul 15, 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 the backport Label PR's that are backports to older release branches label Jul 15, 2025
Copy link

blathers-crl bot commented Jul 15, 2025

✅ PR #150286 is compliant with backport policy

Confidence: high
Backward compatible: true
Explanation: The PR changes a file changefeed_test.go which is a test file, indicated by the _test.go suffix. According to the CockroachDB backport policy, changes that only affect non-production code, tests, or development tools are exempt from the critical bug justification or feature flag requirements. Additionally, the PR contains a release justification stating it is a 'test-only fix', which aligns with the policy for test-only changes not being subject to the usual requirements.

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

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@blathers-crl blathers-crl bot added the backport-test-only Used to denote the backport has only non-production changes label Jul 15, 2025
@andyyang890 andyyang890 merged commit 85a5d5a into cockroachdb:release-25.1 Jul 16, 2025
16 checks passed
@andyyang890 andyyang890 deleted the backport25.1-150215 branch July 16, 2025 13:40
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 backport-test-only Used to denote the backport has only non-production changes target-release-25.1.10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants