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

Migrate nlohmann json to tencent rapidjson #110

Open
testillano opened this issue Nov 29, 2023 · 1 comment
Open

Migrate nlohmann json to tencent rapidjson #110

testillano opened this issue Nov 29, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@testillano
Copy link
Owner

Is your feature request related to a problem? Please describe.

It is not strictly a problem use nlohmann json library, as its performance is very good.
But in general, rapidjson is faster and more efficient, so, we should try to give a chance for this migration.
Check benchmarking tests here: https://github.com/miloyip/nativejson-benchmark

@testillano testillano added the enhancement New feature or request label Nov 29, 2023
@testillano
Copy link
Owner Author

Consider also to implement an LRU cache when building rapidjson::Pointer objects.
You could use this one, which works right: https://github.com/lamerman/cpp-lru-cache/blob/master/include/lrucache.hpp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant