-
Notifications
You must be signed in to change notification settings - Fork 48
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
Support for node streams? #680
Comments
|
Looking at the library I don't think it would be easy to target node's stream api because it is async and callback based, while I see an experimental web streams api, but it's not the same as nodes. https://developer.mozilla.org/en-US/docs/Web/API/Streams_API I'm going to try reading the stream in a |
After some initial success I've hit a roadblock |
i would say iterators are the common factor, and it's easy to create a node stream from it stream.Readable.from(iterator) think there was some proposal also about |
@jimmywarting Can you please elaborate maybe with an example of using an iterator for reader API? Did you mean to provide an iterator for DOM values (It does mean materializing Ion data to DOM values)? |
Does this library support, or plan to support, node's stream module? The
makeReader
function only accepts buffers. That requires reading the whole ion document into memory before beginning to parse.The text was updated successfully, but these errors were encountered: