search

aviggiano / redis-roaring

Roaring Bitmaps for Redis
MIT License
345 stars 55 forks source link

fix core on r.optimize #83 #84

Closed arthurkiller closed 3 years ago

aviggiano commented 3 years ago

This is great, thanks for contributing

Could you please add an integration test with this error?

Here https://github.com/aviggiano/redis-roaring/blob/master/tests/integration_1.sh

arthurkiller commented 3 years ago

I will do this later.

BTW, why we did not try to use tcl test instead? just like what redis do

aviggiano commented 3 years ago

Yea that would be great, but when I did do these first tests I was just looking for something quick n dirty Unfortunately it doesn't scale very well, so migrating to a more robust framework would be better