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

[C++] CSV should include line/row numbers in parsing error messages #28423

Closed
asfimport opened this issue May 6, 2021 · 1 comment
Closed

Comments

@asfimport
Copy link

Currently if an error occurs while parsing a CSV file the absolute line/row number is not part of the error message. Without the line number it can be very hard to determine where the incorrect value is especially in vary large files.

It would be good if the reader and parser could keep track of the row number and maybe line number when new lines are allowed in values so that they can be included in error message.

Reporter: Nate Clark / @n3world
Assignee: Nate Clark / @n3world

PRs and other links:

Note: This issue was originally created as ARROW-12675. Please see the migration documentation for further details.

@asfimport
Copy link
Author

Antoine Pitrou / @pitrou:
Issue resolved by pull request 10321
#10321

@asfimport asfimport added this to the 5.0.0 milestone Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant