Skip to content
This repository has been archived by the owner on Jan 16, 2019. It is now read-only.

Streaming / Incomplete Support #36

Open
argl opened this issue Nov 28, 2015 · 3 comments
Open

Streaming / Incomplete Support #36

argl opened this issue Nov 28, 2015 · 3 comments

Comments

@argl
Copy link

argl commented Nov 28, 2015

The documentation mentions the upcoming streaming support. Is this still on the list? Or should one use jsx directly from Elixir for now?

@talentdeficit
Copy link
Owner

exjsx is in need of some attention. i'll try to add streaming support sometime this week and release a new version

@argl
Copy link
Author

argl commented Nov 30, 2015

Great! I am dabbling with a CouchDB client and streaming JSON allows the changes feed to be parsed correctly, plus possibly streaming views.

@ku1ik
Copy link

ku1ik commented Jul 20, 2017

It would be fantastic to be able to use exjsx in incremental mode.

iex(1)> JSX.decode("[]", [:stream])
{:error, :badarg}

I did some digging. It seems this is the place where incremental support is explicitly killed: https://github.com/talentdeficit/exjsx/blob/master/lib/jsx.ex#L16
I understand it was done because the feature probably needs more wrapping here and there so it was easier to just disable it and come up with proper impl later.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants