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

Jsapar internal #27

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

stenix71
Copy link

@stenix71 stenix71 commented Mar 23, 2019

Hi
I have added the JSaPar parser to the comparison but I have not updated the readme with the result.
You find the parser project at https://github.com/org-tigris-jsapar/jsapar
I added two flavors of the parser, feel free to remove or comment out any of the two. The fastest one (JSaPar internal) does only the string splitting part while the slower (JSaPar) does a lot of additional stuff matching the input to a schema so that each cell can be retrieved by a name and by correct type instead of only as strings by it's position.
Feel free to squash the commits while merging if it is possible in GitHub.
I must add that this comparison project really triggered me to look into performance while parsing. The result was JSaPar version 2.0.1 where a lot of the internals was re-written in order to improve performance. Thanks.

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

Successfully merging this pull request may close these issues.

1 participant