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

Review benefits of async / parallel processing #3

Open
mawosoft opened this issue Nov 30, 2021 · 0 comments
Open

Review benefits of async / parallel processing #3

mawosoft opened this issue Nov 30, 2021 · 0 comments
Labels
performance question Further information is requested

Comments

@mawosoft
Copy link
Owner

Benchmark async vs sync parsing and parallel processing for single and multiple files.

  • None of the XmlReader.ReadToXxx() functions support async processing, plus there is extra overhead in the internal async implementation. OTOH, most of the ReadToXxx() are simple enough to replicate with use of ReadAsync() instead of Read().
    Since a single report file seems the most common use case, we keep this sync for now.
  • Parallel processing of a single file is not possible with XmlReader. Parallel processing of XDocument/XElement via Linq is really fast, but requires the entire file to be loaded as DOM, which not only negates those performance gains, but is about three times slower than sync processing with XmlReader.
@mawosoft mawosoft added question Further information is requested performance labels Nov 30, 2021
@mawosoft mawosoft changed the title Analyze benefits of async / parallel processing Review benefits of async / parallel processing Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant