search

rpkilby / jsonfield

A reusable Django model field for storing ad-hoc JSON data
MIT License
1.16k stars 271 forks source link

Move to simdjson praser #256

Open mkalioby opened 3 years ago

mkalioby commented 3 years ago

Hi,

Thanks for the great library, I'm using this since 2015.

I'm suggesting to change the JSON praser to simdjson (https://github.com/simdjson/simdjson) as it is much faster (based on stats on the repo), the library has a python binding (https://github.com/TkTech/pysimdjson), so can you tell me the feasibility of doing so? and i can help of you like.

Thanks for the great library again.

rpkilby commented 3 years ago

Hi @mkalioby. Changing the default json implementation won't happen (compatibility assurances, extra dependency), however it is worth considering making the implementation swappable.

mkalioby commented 3 years ago

Thanks @rpkilby , how do you suggest to do that?