BigInt cache serialisation error

appy-one / acebase

A fast, low memory, transactional, index & query enabled NoSQL database engine and server for node.js and browser with realtime data change notifications

MIT License

492 stars 27 forks source link

BigInt cache serialisation error #141

Closed Mitsichury closed 2 years ago

Mitsichury commented 2 years ago

Hi :) When filtering on an indexed BigInt value, it throws an error as a big int as no default way to be serialized. The proposed solution work on serialisation but it does not work at all querying results. I opened a draft PR with an example : https://github.com/appy-one/acebase/pull/140 Hope it can help you :) Thank you !

appy-one commented 2 years ago

Thanks! I'll take a look at your PR and what else needs work for queries to work with BigInts

appy-one commented 2 years ago

I'll publish a new version later today with the fix!

appy-one commented 2 years ago

Fix has been published in v1.24.0, let me know if it works! Thanks again for your PR and done research!