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

feat(copy): support copy from 'x.csv' grammar #554

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

infdahai
Copy link

@infdahai infdahai commented Mar 28, 2023

issue: #404

  1. just add parser's support for this statement.

the output of binder_test

Exception Type :: Not implemented
Message :: T_CopyStmt
unknown file: Failure
C++ exception with description "T_CopyStmt" thrown in the test body.

@skyzh
Copy link
Member

skyzh commented Jun 22, 2023

I didn't realize copy from / copy to are not standard Postgres SQL. But given we are actually using DuckDB's parser, I'd prefer keep this repo as-is DuckDB. Do you know which commit you are using from DuckDB?

@infdahai
Copy link
Author

I will test it in the latest branch after a few days. I forgot the origin commit.

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.

2 participants