Skip to content

Mark CopyIn() and CopyInSchema() as deprecated#1279

Merged
arp242 merged 1 commit intomasterfrom
copy
Mar 11, 2026
Merged

Mark CopyIn() and CopyInSchema() as deprecated#1279
arp242 merged 1 commit intomasterfrom
copy

Conversation

@arp242
Copy link
Collaborator

@arp242 arp242 commented Mar 11, 2026

These are just simple string-concat query builders, and don't really do anything special. These two are identical:

tx.Prepare(CopyIn("temp", "num", "text", "blob", "nothing"))

tx.Prepare(`copy temp (num, text, blob, nothing) from stdin`)

IMO having a (fairly incomplete) query builder doesn't really add much.

Closes #688

These are just simple string-concat query builders, and don't really do
anything special. These two are identical:

	tx.Prepare(CopyIn("temp", "num", "text", "blob", "nothing"))

	tx.Prepare(`copy temp (num, text, blob, nothing) from stdin`)

IMO having a (fairly incomplete) query builder doesn't really add much.

Closes #688
@arp242 arp242 merged commit 2a2614c into master Mar 11, 2026
13 checks passed
@arp242 arp242 deleted the copy branch March 11, 2026 23:51
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.

copy.go: Missing field to add options

1 participant