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.

Sign up for GitHub

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

Jump to bottom

copy_dm_to() should support AI PKs #1725

Open
TSchiefer opened this issue Jan 4, 2023 · 2 comments · May be fixed by #2022
Open

copy_dm_to() should support AI PKs #1725

TSchiefer opened this issue Jan 4, 2023 · 2 comments · May be fixed by #2022
Milestone
copy

Comments

@TSchiefer
Copy link
Member

(see #1654 (comment): copy_dm_to() should eventually be able to create and fill data when one or more tables with AI PKs are to be copied)

still left to do after #1689, #1698 & co.
@krlmlr
Copy link
Collaborator

krlmlr commented Jan 8, 2023

Low priority, users can copy the ptype and then do dm_rows_append() in many cases.

@krlmlr krlmlr added this to the copy milestone Aug 20, 2023
@krlmlr
Copy link
Collaborator

krlmlr commented Oct 7, 2023

What is the expected behavior here? For MSSQL, we issue SET IDENTITY_INSERT OFF before inserting to make sure that we can bring our own values for the autoincrement primary key.

@krlmlr krlmlr linked a pull request Oct 9, 2023 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
copy
2 participants
@krlmlr @TSchiefer