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

expose createReadStream / createWriteStream #18

Open
1N50MN14 opened this issue Feb 23, 2016 · 2 comments
Open

expose createReadStream / createWriteStream #18

1N50MN14 opened this issue Feb 23, 2016 · 2 comments

Comments

@1N50MN14
Copy link

Hi @ksperis thank you very much for making this available in node.js! ;)

Would it be possible for you wrap / expose a node.js style createReadStream and createWriteStream around io_read() and io_write()?

This would make it easier to do something like:
bufferStream.pipe(ioctx.createWriteStream('obj-id'))

@1N50MN14
Copy link
Author

oh I just saw in your test file ioctx.createReadStream is already exposed, apologies about that I didn't see it in the README. Would it be possible to expose createWriteStream in the same manner?

@ksperis
Copy link
Owner

ksperis commented Mar 1, 2016

hi @1N50MN14,
I added a simple implementation of createWrtiteStream 5bf1a53 .
I think this should be more tested, because the calls are asynchronous and we must be sure that writes are well ordered.

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

2 participants