surrealdb / surrealdb.js

SurrealDB SDK for JavaScript
https://surrealdb.com
Apache License 2.0
287 stars 47 forks source link

CBOR performance improvements #304

Closed kearfy closed 1 month ago

kearfy commented 1 month ago

Thank you for submitting this pull request! We appreciate you spending the time to work on these changes.

What is the motivation?

As reported in Discord, our custom CBOR implementation had some serious performance implications

What does this change do?

Made some improvements to the encoder, decoder, writer and reader, making the CBOR implementation even faster than the original library that we used.

What is your testing strategy?

GitHub CI. Benchmark tests to be added soon to prevent this sort of performance decline from happening in the future.

Is this related to any issues?

No

Have you read the Contributing Guidelines?