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

Fix create table construction #47

Merged
merged 4 commits into from
Feb 5, 2021
Merged

Conversation

karoliskascenas
Copy link
Contributor

STORED AS should be at the end of the create table statement.

Repository owner deleted a comment from codecov-io Feb 3, 2021
@ianmcook
Copy link
Owner

ianmcook commented Feb 3, 2021

Thanks @karoliskascenas. The implyr CI checks are broken right now because the VM they were running on was terminated. As soon as I get that working again I will recheck this and merge your PR.

`STORED AS` should be at the end of the create table statement.
@ianmcook
Copy link
Owner

ianmcook commented Feb 5, 2021

@karoliskascenas I'm going to push some more commits into this PR to fix the handling of field_terminator and line_terminator

@ianmcook ianmcook merged commit 7da147e into ianmcook:master Feb 5, 2021
@karoliskascenas karoliskascenas deleted the patch-1 branch February 8, 2021 06:56
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.

None yet

2 participants