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

crosscluster/logical: properly handle incorrect targets in CREATE stmt #137975

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

msbutler
Copy link
Collaborator

Previously, if the schema or database were missing from a destination target table in CREATE LOGICALLY REPLICATED TABLE, the planHook would panic.

This patch also beefs up testing around user specified replication targets.

Fixes #137745

Release note: none

@msbutler msbutler self-assigned this Dec 25, 2024
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Previously, if the schema or database were missing from a destination target
table in CREATE LOGICALLY REPLICATED TABLE, the planHook would panic.

This patch also beefs up testing around user specified replication targets.

Fixes cockroachdb#137745

Release note: none
@msbutler msbutler force-pushed the butler-create-ldr-syntax-bug branch from 6fd60cc to 7146cec Compare December 25, 2024 01:19
@msbutler msbutler marked this pull request as ready for review December 25, 2024 14:33
@msbutler msbutler requested a review from a team as a code owner December 25, 2024 14:33
@msbutler msbutler requested review from dt and jeffswenson and removed request for a team December 25, 2024 14:33
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.

sql/tests: TestRandomSyntaxGeneration failed [nil pointer in createLogicalReplicationStreamPlanHook]
2 participants