dice icon indicating copy to clipboard operation
dice copied to clipboard

#1031: Bloom Filter - Migrated `bf.add`, `bf.reserve`, `bf.exists` & `bf.info` to store_eval

Open apoorvyadav1111 opened this issue 4 months ago • 0 comments

Fixes: #1031 #1110

For migration commands, please consider this a template on how to tackle the issue. I have divided the tasks into smaller goals and made an individual commit.

Condition Return Value
Migrating eval to store eval ebba934
Modify unit test with new response format d4ae3eb
Migrate integration tests from async to resp 81ff86e
Add/update integration in websocket c1764b4
Add/update integration in HTTP 42fd50c
Ensure all integration test pass 42fd50c
Update/Add docs for the commands

Checklist

  • [x] Migrated the evalXXX function with the latest definition
  • [x] Update or add unit tests for the new implementation.
  • [x] All unit tests pass successfully.
  • [x] Ensure all integration tests pass successfully.
  • [x] Move Integration tests for the respective commands under the RESP integration tests directory from the Async directory
  • [ ] Please validate that the documentation for the respective commands is up to date. If not then consider adding them.

apoorvyadav1111 avatar Oct 16 '24 18:10 apoorvyadav1111