Skip to content
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

Support passing args to MODULE LOAD #2918

Merged
merged 6 commits into from
Feb 22, 2022

Conversation

Avital-Fine
Copy link
Contributor

closes #2904

@codecov-commenter
Copy link

Codecov Report

Merging #2918 (30eecca) into master (0c36557) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2918      +/-   ##
============================================
- Coverage     55.06%   55.04%   -0.03%     
- Complexity     3157     3158       +1     
============================================
  Files           195      195              
  Lines         12091    12098       +7     
  Branches        691      691              
============================================
+ Hits           6658     6659       +1     
- Misses         5205     5212       +7     
+ Partials        228      227       -1     
Impacted Files Coverage Δ
src/main/java/redis/clients/jedis/Jedis.java 84.53% <0.00%> (-0.37%) ⬇️
src/main/java/redis/clients/jedis/Module.java 33.33% <ø> (ø)
...nts/jedis/providers/ClusterConnectionProvider.java 81.81% <0.00%> (+3.63%) ⬆️

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 0c36557...30eecca. Read the comment docs.

@Avital-Fine Avital-Fine requested a review from sazzad16 February 21, 2022 15:32
Avital-Fine and others added 3 commits February 22, 2022 19:43
Co-authored-by: M Sazzadul Hoque <7600764+sazzad16@users.noreply.github.com>
Co-authored-by: 小令童鞋 <lingzhaohui@zeekling.cn>
@Avital-Fine Avital-Fine requested a review from sazzad16 February 22, 2022 20:24
@sazzad16 sazzad16 added this to the 4.2.0 milestone Feb 22, 2022
@sazzad16 sazzad16 merged commit e0d762e into redis:master Feb 22, 2022
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.

Add support for redis command: MODULE LOAD - [arg [arg ...]]
4 participants