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.

Sign up for GitHub

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

Jump to bottom

Support for non-Delimited file types in Redshift #39

Closed
fdosani opened this issue Mar 9, 2019 · 2 comments · Fixed by #153
Closed

Support for non-Delimited file types in Redshift #39

fdosani opened this issue Mar 9, 2019 · 2 comments · Fixed by #153
Assignees
@fdosani
Labels
enhancement New feature or request nice to have (non-critical)

Comments

@fdosani
Copy link
Member

fdosani commented Mar 9, 2019

https://docs.aws.amazon.com/redshift/latest/dg/copy-parameters-data-format.html
@fdosani fdosani added the enhancement New feature or request label Mar 9, 2019
@fdosani fdosani self-assigned this Mar 9, 2019
@jfyu
Copy link
Contributor

jfyu commented Jun 23, 2022 •
edited
Loading

Can I add a +1 to this issue 😂 currently running into this now. It seems to me that you just have to not include DELIMITER as an option? i.e. make this line optional only if deliminter is not None ?

Also now I'm using locopy because Redshift really has crappy python connector....

This is because writing CSV had an issue if a lot of my columns are null, and I run into Error Tokenizing Data issue when read_csv

@fdosani
Copy link
Member Author

fdosani commented Jun 23, 2022

@jfyu happy to accept a PR too if you have some time 😄

@jfyu jfyu mentioned this issue Jun 23, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fdosani fdosani

Labels
enhancement New feature or request nice to have (non-critical)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

make delimiter optional jfyu/locopy
2 participants
@fdosani @jfyu