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

SQL: support LOAD DATA INFILE Statement #443

Closed
dl239 opened this issue Sep 22, 2021 · 3 comments · Fixed by #554
Closed

SQL: support LOAD DATA INFILE Statement #443

dl239 opened this issue Sep 22, 2021 · 3 comments · Fixed by #554
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@dl239
Copy link
Collaborator

dl239 commented Sep 22, 2021

desc: batch import data into OpenMLDB
format: import table file_path
ex: import demo_table1 file:///OpenMLDB/cli/examples/taxi_tour_parquet_all/raw_data.csv;

Desc: support LOAD DATA INFILE 'file_name' SQL Statement like MySQL
E.g: LOAD DATA INFILE 'hello.csv' INTO TABLE t1;

@dl239 dl239 added the enhancement New feature or request label Sep 22, 2021
@dl239 dl239 added this to the v0.4 milestone Sep 22, 2021
@dl239 dl239 modified the milestones: v0.4, v0.3 Sep 24, 2021
@aceforeverd aceforeverd self-assigned this Sep 27, 2021
@aceforeverd
Copy link
Collaborator

different with import statements e.g mysql's https://dev.mysql.com/doc/refman/8.0/en/import-table.html

@aceforeverd
Copy link
Collaborator

As discussed in #494 , this import feature should better write as LOAD DATA INFILE, since it do not import schema but just data. Also documented by MySQL:

LOAD DATA is the complement of SELECT ... INTO OUTFILE.

which is more suitable with #455

@aceforeverd aceforeverd changed the title Add import command SQL: support LOAD DATA INFILE Statement Oct 11, 2021
@vagetablechicken
Copy link
Collaborator

vagetablechicken commented Oct 18, 2021

load data supports OPTIONS, ref #455 (comment)
Currently, we can support none, but should warn it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants