alpaca-trade-api-csharp icon indicating copy to clipboard operation
alpaca-trade-api-csharp copied to clipboard

Switch to new JSON handling library

Open OlegRa opened this issue 5 years ago • 4 comments

.NET Core 3.0 and .NET Standard 2.1 provides a new built-in set of classes for fast JSON handling in .NET - this is a new recommended way for handling JSON streams in case of forward-only processing. We can enable this support in a new major SDK release after implementing other .NET Core 3.0 related changes.

OlegRa avatar Nov 15 '19 14:11 OlegRa

After analyzing all the pros and cons of this switch and required amount changes in the source code I've decided to postpone such critical change till next major release.

OlegRa avatar Dec 10 '19 21:12 OlegRa

We have too many features in our deserialization code which heavily depends on JSON.NET library features and now I don't see any real reason for rewriting all this stuff using a new library. I'll not close it but now it has no fixed milestone. If someone will do it or provide real reason why should we implement it I can return back to this task.

OlegRa avatar Mar 04 '20 16:03 OlegRa

We need this one for issue #144 (at least for the REST part of this issue).

OlegRa avatar Oct 06 '20 17:10 OlegRa

Another good reason to postpone it till .NET 6 release: no-reflection solution via source generators!

OlegRa avatar Jul 26 '21 10:07 OlegRa