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

define generated column on CDC table #16024

Closed
yuhao-su opened this issue Mar 31, 2024 · 6 comments · Fixed by #19112
Closed

define generated column on CDC table #16024

yuhao-su opened this issue Mar 31, 2024 · 6 comments · Fixed by #19112
Assignees
Milestone

Comments

@yuhao-su
Copy link
Contributor

yuhao-su commented Mar 31, 2024

          this feature does not seem to work with Postgres CDC table
create table courses (*) from pg_mydb table 'public.courses';
ERROR:  Failed to run the query
Caused by these errors (recent errors listed first):
  1: gRPC request to meta service failed: Internal error
  2: failed to validate CDC table
  3: source cannot pass validation: INVALID_ARGUMENT: Primary key mismatch

Originally posted by @mingfang in #14644 (comment)

@github-actions github-actions bot added this to the release-1.8 milestone Mar 31, 2024
@fuyufjh fuyufjh modified the milestones: release-1.8, release-1.9 Apr 8, 2024
@StrikeW
Copy link
Contributor

StrikeW commented Apr 10, 2024

Is this issue the same problem as #14693 ?

@yuhao-su
Copy link
Contributor Author

Is this issue the same problem as #14693 ?

Those are different issue

@yuhao-su yuhao-su changed the title define generated column on CDC source define generated column on CDC table Apr 22, 2024
@st1page
Copy link
Contributor

st1page commented Apr 26, 2024

Also, the old CDC connector can not support that too.

dev=> create table tt1 (
  pk int,
  v int, v1 int as v+v, 
  PRIMARY KEY (pk)
) with (
  connector = 'postgres-cdc',
  hostname = 'localhost',
  port = '5432',
  username = 'postgres',
  password = '123456',
  database.name = 'postgres',
  schema.name = 'public',
  table.name = 't',
  slot.name = 'tt1_slot',
);
ERROR:  Failed to run the query

Caused by these errors (recent errors listed first):
  1: gRPC request to meta service failed: Internal error
  2: failed to create source worker
  3: failed to create SplitEnumerator
  4: source cannot pass validation
  5: INVALID_ARGUMENT: The publication 'rw_publication' does not cover all columns of the table 'public.t'

@st1page
Copy link
Contributor

st1page commented Apr 29, 2024

Also, the old CDC connector can not support that too.

dev=> create table tt1 (
  pk int,
  v int, v1 int as v+v, 
  PRIMARY KEY (pk)
) with (
  connector = 'postgres-cdc',
  hostname = 'localhost',
  port = '5432',
  username = 'postgres',
  password = '123456',
  database.name = 'postgres',
  schema.name = 'public',
  table.name = 't',
  slot.name = 'tt1_slot',
);
ERROR:  Failed to run the query

Caused by these errors (recent errors listed first):
  1: gRPC request to meta service failed: Internal error
  2: failed to create source worker
  3: failed to create SplitEnumerator
  4: source cannot pass validation
  5: INVALID_ARGUMENT: The publication 'rw_publication' does not cover all columns of the table 'public.t'

fixed in #16522

Copy link
Contributor

github-actions bot commented Jul 3, 2024

This issue has been open for 60 days with no activity.
If you think it is still relevant today, and needs to be done in the near future, you can comment to update the status, or just manually remove the no-issue-activity label.
You can also confidently close this issue as not planned to keep our backlog clean. Don't worry if you think the issue is still valuable to continue in the future. It's searchable and can be reopened when it's time. 😄

@st1page
Copy link
Contributor

st1page commented Dec 2, 2024

open a more detailed issue for cdc table with auto schema mapping #19643

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

Successfully merging a pull request may close this issue.

4 participants