-
Notifications
You must be signed in to change notification settings - Fork 102
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
CSVExternalSort drops rows when using CSVFormat with .format option #44
Comments
why? csv supports newlines...how do you use double quotes and newlines in the file?
…On Sun, Mar 19, 2023 at 2:31 PM Daniel Lemire ***@***.***> wrote:
That's not a valid CSV input...
"author","title"
"Dan Simmons","Hyperion"
"Dan Simmons","Hyperion"
"Dan Simmons","Hyperion"
"Dan Simmons","Hyperion"
"Dan Simmons","Hyperion"
"Dan Simmons","Hyperion"
"Douglas Adams","The Hitchhiker's Guide to the Galaxy"
"Douglas Adams","The Hitchhiker's Guide to the Galaxy"
"Douglas Adams","The Hitchhiker's Guide to the Galaxy
hello"
"Douglas Adams","The Hitchhiker's Guide to the Galaxy"
"Douglas Adams","The Hitchhiker's Guide to the Galaxy"
"Douglas Adams","The Hitchhiker's Guide to the Galaxy"
"Douglas Adams","The Hitchhiker's Guide to the Galaxy"
"Dan ""The Man"" Simmons","Hyperion"
—
Reply to this email directly, view it on GitHub
<#44 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AQUKIE5SUHELOZSQJSJ2PE3W45GHXANCNFSM6AAAAAAV7Y3MZM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
I thought it was invalid because of all the quotes. You are correct, it is valid. |
I have added a test to the project corresponding to your use case. I am closing this issue, if you believe that there is a bug, please provide a reproducible test case. Ideally, you should provide a pull request. |
Repository owner
locked as resolved and limited conversation to collaborators
Mar 19, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
ExternalSorting.zip
Attached a project demonstrating issue ....
The text was updated successfully, but these errors were encountered: