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

Weird error message when using dolt import #275

Closed
oscarbatori opened this issue Dec 28, 2019 · 3 comments · Fixed by #363 or #366
Closed

Weird error message when using dolt import #275

oscarbatori opened this issue Dec 28, 2019 · 3 comments · Fixed by #363 or #366
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers
Projects

Comments

@oscarbatori
Copy link
Contributor

[oscarbatori:scratch] >> dolt table import people --pk=first -c test_data.csv
Data already exists.  Use -f to overwrite.

Should say "Table already exists".

@oscarbatori oscarbatori added documentation Improvements or additions to documentation good first issue Good for newcomers labels Dec 28, 2019
@oscarbatori oscarbatori added this to Backlog in Dolt Dec 28, 2019
@timsehn timsehn reopened this Feb 4, 2020
@timsehn
Copy link
Sponsor Contributor

timsehn commented Feb 4, 2020

I think the export table path was changed by the PR above not the import path.

@timsehn
Copy link
Sponsor Contributor

timsehn commented Feb 4, 2020

@andrew-wm-arthur is gonna fix and make a bats test.

@timsehn
Copy link
Sponsor Contributor

timsehn commented Feb 4, 2020

So it turns out that error code was used on both the import and export path. So, when we merged the PR one of our integration tests failed. @andrew-wm-arthur is going to make two different paths, one for export and one for import.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers
Projects
No open projects
Dolt
  
Backlog
2 participants