RedisBloom / JRedisBloom

Java Client for RedisBloom probabilistic module
https://redisbloom.io
BSD 2-Clause "Simplified" License
153 stars 33 forks source link

Implements Cuckoo Filter RedisBloom Commands #31

Closed bsbodden closed 3 years ago

bsbodden commented 3 years ago
codecov[bot] commented 3 years ago

Codecov Report

Merging #31 (6adeb21) into master (f75c824) will decrease coverage by 33.42%. The diff coverage is 0.45%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #31       +/-   ##
===========================================
- Coverage   79.26%   45.84%   -33.43%     
===========================================
  Files           6        9        +3     
  Lines         299      517      +218     
  Branches       34       69       +35     
===========================================
  Hits          237      237               
- Misses         43      261      +218     
  Partials       19       19               
Impacted Files Coverage Δ
...ain/java/io/rebloom/client/cf/CFInsertOptions.java 0.00% <0.00%> (ø)
...in/java/io/rebloom/client/cf/CFReserveOptions.java 0.00% <0.00%> (ø)
.../main/java/io/rebloom/client/cf/CuckooCommand.java 0.00% <0.00%> (ø)
src/main/java/io/rebloom/client/Client.java 35.41% <0.72%> (-46.32%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f75c824...6adeb21. Read the comment docs.

ashtul commented 3 years ago

This is well overdue... :+1:

bsbodden commented 3 years ago

@sazzad16 @gkorland I believe I've addressed all the requests. Let me know if there is anything missing.

bsbodden commented 3 years ago

@sazzad16 @gkorland I believe I've addressed all the requests. Let me know if there is anything missing.

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 45bda36374b6d167f7ce41cc4247d9547a9c07d9 into 2fee11c034bd2de27496d26386ee8be03a43716e - view on LGTM.com

new alerts: