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

add new parser based on Utf8StreamReader for comparison #68

Open
DrJakiellSWO opened this issue May 27, 2024 · 3 comments
Open

add new parser based on Utf8StreamReader for comparison #68

DrJakiellSWO opened this issue May 27, 2024 · 3 comments

Comments

@DrJakiellSWO
Copy link

hello,
can you in your benchmarks add new parser based on Utf8StreamReader ?

@joelverhagen
Copy link
Owner

Feel free to open a PR and I will rerun the benchmarks!

@ksysiekj
Copy link

@joelverhagen cant be done as Utf8StreamReader supports only async read. Can't use it as the ICsvReader.GetRecords method is synch one

@joelverhagen
Copy link
Owner

Feel free to enhance the interface for async and add an associated async benchmark. I think I just had a sync method until now since all of the CSV libraries so far have had a sync path.

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

No branches or pull requests

3 participants