Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

Change JSON formatters to be async #6002

Closed
kichalla opened this issue Mar 22, 2017 · 6 comments
Closed

Change JSON formatters to be async #6002

kichalla opened this issue Mar 22, 2017 · 6 comments

Comments

@kichalla
Copy link
Member

Latest JSON.Net package supports async reading and writing.
http://james.newtonking.com/archive/2017/03/21/json-net-10-0-release-1-async-performance-documentation-and-more

cc @DamianEdwards @pranavkm @rynowak

@kichalla
Copy link
Member Author

The async apis are not available on the JsonSerializer type (which we use here and here) yet, so looks like we cannot move to async yet. Async support is available on JsonWriter and JsonReader but that doesn't help us here.

@kichalla
Copy link
Member Author

@JamesNK Is support for exposing async api on JsonSerializer part of your future plans?

@JamesNK
Copy link
Member

JamesNK commented May 24, 2017

Maybe one day but not right now. It will be a lot of work

@kichalla
Copy link
Member Author

Thanks @JamesNK .

Closing this issue for now as there is no action item here.

@rynowak rynowak reopened this May 24, 2017
@rynowak
Copy link
Member

rynowak commented May 24, 2017

There is definitely an action item here for us, it just might not be to make the formatters async.

@kichalla
Copy link
Member Author

kichalla commented Jun 6, 2017

f1dd475

@kichalla kichalla closed this as completed Jun 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants