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
release-22.2: sql: fix CREATE TABLE AS schema change job description #107402
Conversation
Add the invalid type to the `type reference must implement NodeFormatter` error to help with debugging. Release note: None
Refactor `newTableDescIfAs` slightly to make it clear that `p.Defs` iteration only happens if `p.Defs` is not empty. Release note: None
Fixes #107364 This changes the CREATE TABLE AS schema change job description to no longer incorrectly include column types. For example `CREATE TABLE movr.public.t2 (id INT8) AS SELECT * FROM movr.public.t;` becomes `CREATE TABLE movr.public.t2 (id) AS SELECT * FROM movr.public.t;` Release note (bug fix): Fix CREATE TABLE AS schema change job description SQL syntax.
68a8bab
to
b2f81f9
Compare
abffc64
to
10b72aa
Compare
Thanks for opening a backport. Please check the backport criteria before merging:
If some of the basic criteria cannot be satisfied, ensure that the exceptional criteria are satisfied within.
Add a brief release justification to the body of your PR to justify this backport. Some other things to consider:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 1 of 1 files at r1, 1 of 1 files at r2, 3 of 3 files at r3, all commit messages.
Reviewable status: complete! 0 of 0 LGTMs obtained (waiting on @ecwall)
Backport 3/3 commits from #107367 on behalf of @ecwall.
/cc @cockroachdb/release
Fixes #107364
This changes the CREATE TABLE AS schema change job description to no longer incorrectly
include column types.
For example
CREATE TABLE movr.public.t2 (id INT8) AS SELECT * FROM movr.public.t;
becomes
CREATE TABLE movr.public.t2 (id) AS SELECT * FROM movr.public.t;
Release note (bug fix): Fix CREATE TABLE AS schema change job description SQL syntax.
Release justification: CTAS schema change job description fix.