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

Allow an error interface channnel #129

Open
blackmo18 opened this issue Aug 5, 2023 · 1 comment · May be fixed by #131
Open

Allow an error interface channnel #129

blackmo18 opened this issue Aug 5, 2023 · 1 comment · May be fixed by #131

Comments

@blackmo18
Copy link
Contributor

they were use case that we want to process whatever processable row and keep other at bay so only those error rows will be retried after.

Currently kotlin-csv only allow to skips error row.

You have to filter through the whole csv again just to validate what failed.

If there is an error channel, like an injectable lamda functions or whatever mechansm. We can keep the errors seperately
@doyaaaaaken
Copy link
Owner

Thank you for your input.

As you mentioned, there are only a few simple options for dealing with errors currently available.
https://github.com/doyaaaaaken/kotlin-csv#customize

However, even with the current specification, it is possible to deal with complex errors itself.
You can do this by using the following function.
https://github.com/doyaaaaaken/kotlin-csv#read-as-sequence

Since we want kotlin-csv to be a simple specification, we will carefully consider the option to handle complex errors only if we get a lot of requests for it from our users.

@blackmo18 blackmo18 linked a pull request Sep 23, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Skip Notification Event blackmo18/kotlin-csv
2 participants
@doyaaaaaken @blackmo18