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

fs2-data streaming xml parser? #4

Closed
armanbilge opened this issue May 2, 2022 · 7 comments
Closed

fs2-data streaming xml parser? #4

armanbilge opened this issue May 2, 2022 · 7 comments
Labels
question Further information is requested

Comments

@armanbilge
Copy link
Member

Just a random idea worth exploring.

https://github.com/satabin/fs2-data

@armanbilge armanbilge added the question Further information is requested label May 2, 2022
@rossabaker
Copy link
Member

Ooh, that could be neat. There is an incremental one called Aalto that I had my eye on for a while, but never got around to integrating.

This would of course introduce another dependency of unknown stability.

@armanbilge
Copy link
Member Author

I'm partial to fs2-data since it cross-builds for Scala.js 😉 two birds with one stone #3

@armanbilge
Copy link
Member Author

Here's another streaming xml parser.
https://github.com/ScalaWilliam/xs4s

@rossabaker
Copy link
Member

Pros and cons: fs2-data has its own XML parser. xs4s uses the battle-tested Java parser, but requires a java.io.InputStream.

@armanbilge
Copy link
Member Author

The latest release of fs2-data-xml just added support for directly building scala-xml trees. So this is probably extremely easy to implement now.

https://github.com/satabin/fs2-data/releases/tag/v1.4.0

@armanbilge armanbilge linked a pull request Jun 10, 2022 that will close this issue
@rossabaker
Copy link
Member

I did an aalto spike. It's VERY fast, but incomplete, and I think I'm cheating badly with mutable state.

@armanbilge
Copy link
Member Author

@armanbilge armanbilge closed this as not planned Won't fix, can't repro, duplicate, stale Jun 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants