github-starred/acebase

Fork 0

mirror of https://github.com/appy-one/acebase.git synced 2026-05-25 06:02:14 -06:00

[GH-ISSUE #152] Using filter "in" with another filter on indexed bigInt does not return all values #83

New issue

Closed

opened 2026-05-23 08:29:37 -06:00 by gitea-mirror · 7 comments

gitea-mirror commented

2026-05-23 08:29:37 -06:00

Owner

Originally created by @Mitsichury on GitHub (Sep 11, 2022).
Original GitHub issue: https://github.com/appy-one/acebase/issues/152

Originally assigned to: @appy-one on GitHub.

Hi :)

I have trouble getting all values when using indexed BigInt value and combining query filters.
You can find a test that reproduce ths issue here : https://github.com/appy-one/acebase/pull/151/files
Hope it helps you, thank you for your amazing app :)

Best regards,
Mitsi.

Originally created by @Mitsichury on GitHub (Sep 11, 2022). Original GitHub issue: https://github.com/appy-one/acebase/issues/152 Originally assigned to: @appy-one on GitHub. Hi :) I have trouble getting all values when using indexed BigInt value and combining query filters. You can find a test that reproduce ths issue here : https://github.com/appy-one/acebase/pull/151/files Hope it helps you, thank you for your amazing app :) Best regards, Mitsi.

gitea-mirror

2026-05-23 08:29:37 -06:00

closed this issue
added the
bug

dependencies
labels

gitea-mirror commented

2026-05-23 08:29:41 -06:00

Author

Owner

@appy-one commented on GitHub (Sep 11, 2022):

Thanks Mitsi, after some digging, it appears to be a bug in the serialization of BigInts to binary format and/or vice versa. I'll have to fix that algorithm.

@appy-one commented on GitHub (Sep 11, 2022): Thanks Mitsi, after some digging, it appears to be a bug in the serialization of BigInts to binary format and/or vice versa. I'll have to fix that algorithm.

gitea-mirror commented

2026-05-23 08:29:42 -06:00

Author

Owner

@appy-one commented on GitHub (Sep 12, 2022):

Fix has been published in acebase@1.24.2 and acebase-server@1.13.1

Note that you'll have to update the BigInt values in your db because some have been stored incorrectly!

@appy-one commented on GitHub (Sep 12, 2022): Fix has been published in acebase@1.24.2 and acebase-server@1.13.1 Note that you'll have to update the BigInt values in your db because some have been stored incorrectly! [![Sponsor AceBase](https://user-images.githubusercontent.com/26569719/168233053-8e56b243-4140-40ab-9a30-4cb3cc149bfe.svg)](https://github.com/sponsors/appy-one) [![Spread the word](https://user-images.githubusercontent.com/26569719/169265089-3d593555-e1ad-4390-986b-877ac2c38a47.svg)](https://twitter.com/intent/tweet?button=&url=https://github.com/appy-one/acebase&text=I'm+using+@AcebaseRealtime+in+my+project+to+make+my+life+easier!&button=) [![contribute](https://user-images.githubusercontent.com/26569719/169265318-30c4c6a5-7c89-46a0-a7a2-ef433a8192f4.svg)](https://github.com/appy-one/acebase#contributing)

gitea-mirror commented

2026-05-23 08:29:44 -06:00

Author

Owner

@Mitsichury commented on GitHub (Sep 30, 2022):

Hi :)
Sorry for the late answer, I've updated to the last version 1.24.4, my tests still don't work if I create indexes on Bigint and I apply filters on it. But it works really great without indexes. I don't know why for now because your unit tests are green, I'll try to reproduce the issue.
Thank you for your amazing work :) !

@Mitsichury commented on GitHub (Sep 30, 2022): Hi :) Sorry for the late answer, I've updated to the last version 1.24.4, my tests still don't work if I create indexes on Bigint and I apply filters on it. But it works really great without indexes. I don't know why for now because your unit tests are green, I'll try to reproduce the issue. Thank you for your amazing work :) !

gitea-mirror commented

2026-05-23 08:29:46 -06:00

Author

Owner

@Mitsichury commented on GitHub (Sep 30, 2022):

I've found how to reproduce it !
I've modified a test to make it close to my usage : https://github.com/appy-one/acebase/pull/159/files

Empty db
Creating indexes
Adding some data by unit call instead of set collection (like if I did it with http requests)
Requesting data

I don't know if it's a bug or a misusage from me.

Thank you,
Mitsi.

@Mitsichury commented on GitHub (Sep 30, 2022): I've found how to reproduce it ! I've modified a test to make it close to my usage : https://github.com/appy-one/acebase/pull/159/files - Empty db - Creating indexes - Adding some data by unit call instead of set collection (like if I did it with http requests) - Requesting data I don't know if it's a bug or a misusage from me. Thank you, Mitsi.

gitea-mirror commented

2026-05-23 08:29:46 -06:00

Author

Owner

@appy-one commented on GitHub (Oct 3, 2022):

Thanks for your PR, I'm on it!

@appy-one commented on GitHub (Oct 3, 2022): Thanks for your PR, I'm on it!

gitea-mirror commented

2026-05-23 08:29:47 -06:00

Author

Owner

@appy-one commented on GitHub (Oct 3, 2022):

Hi Mitsi, I've fixed the issue and released new version v1.24.5
Note that the code in your PR does contain an async issue with forEach, see my comments there.
Let me know if it all works!

@appy-one commented on GitHub (Oct 3, 2022): Hi Mitsi, I've fixed the issue and released new version v1.24.5 Note that the code in your PR does contain an async issue with `forEach`, see my comments there. Let me know if it all works!

gitea-mirror commented

2026-05-23 08:29:48 -06:00

Author

Owner

@Mitsichury commented on GitHub (Oct 4, 2022):

Hi @appy-one thank you for your review and your fix, it's working really good :D
Best regards,
Mitsi.

@Mitsichury commented on GitHub (Oct 4, 2022): Hi @appy-one thank you for your review and your fix, it's working really good :D Best regards, Mitsi.