Skip to content

Bloom commands #1786

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Dec 29, 2021
Merged

Bloom commands #1786

merged 15 commits into from
Dec 29, 2021

Conversation

AvitalFineRedis
Copy link
Contributor

@AvitalFineRedis AvitalFineRedis commented Dec 15, 2021

Description

Describe your pull request here


Checklist

  • Does npm test pass with this change (including linting)?
  • Is the new or changed code fully tested?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

@AvitalFineRedis AvitalFineRedis marked this pull request as ready for review December 15, 2021 17:46
@codecov-commenter
Copy link

codecov-commenter commented Dec 27, 2021

Codecov Report

Merging #1786 (58258dd) into master (b97d18b) will increase coverage by 0.64%.
The diff coverage is 99.14%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1786      +/-   ##
==========================================
+ Coverage   94.01%   94.65%   +0.64%     
==========================================
  Files         316      353      +37     
  Lines        3323     3293      -30     
  Branches      453      409      -44     
==========================================
- Hits         3124     3117       -7     
+ Misses        112       92      -20     
+ Partials       87       84       -3     
Impacted Files Coverage Δ
packages/bloom/lib/commands/cuckoo/SCANDUMP.ts 80.00% <80.00%> (ø)
packages/bloom/lib/commands/bloom/SCANDUMP.ts 83.33% <83.33%> (ø)
packages/bloom/lib/commands/bloom/ADD.ts 100.00% <100.00%> (ø)
packages/bloom/lib/commands/bloom/EXISTS.ts 100.00% <100.00%> (ø)
packages/bloom/lib/commands/bloom/INFO.ts 100.00% <100.00%> (ø)
packages/bloom/lib/commands/bloom/INSERT.ts 100.00% <100.00%> (ø)
packages/bloom/lib/commands/bloom/LOADCHUNK.ts 100.00% <100.00%> (ø)
packages/bloom/lib/commands/bloom/MADD.ts 100.00% <100.00%> (ø)
packages/bloom/lib/commands/bloom/MEXISTS.ts 100.00% <100.00%> (ø)
packages/bloom/lib/commands/bloom/RESERVE.ts 100.00% <100.00%> (ø)
... and 28 more

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 b97d18b...58258dd. Read the comment docs.

@leibale leibale merged commit f93ac04 into master Dec 29, 2021
@leibale leibale deleted the Bloom_commands branch December 29, 2021 16:55
florian-schunk pushed a commit to florian-schunk/node-redis that referenced this pull request Jun 18, 2025
* ft.alter

* bloom commands

* tdigest

* delete tdigest

* uncomment tests

* small changes

* Update MADD.ts

* small changes

* clean code

* Update README.md

* Update README.md

Co-authored-by: leibale <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants