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

Fix Search/Gears test regression #3552

Merged
merged 4 commits into from
Sep 23, 2023

Conversation

sazzad16
Copy link
Collaborator

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 23, 2023

Codecov Report

Patch coverage: 83.76% and project coverage change: +0.24% 🎉

Comparison is base (af92035) 71.23% compared to head (a0e61b5) 71.48%.
Report is 2 commits behind head on master.

❗ Current head a0e61b5 differs from pull request most recent head 1270812. Consider uploading reports for the commit 1270812 to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3552      +/-   ##
============================================
+ Coverage     71.23%   71.48%   +0.24%     
- Complexity     4791     4841      +50     
============================================
  Files           278      287       +9     
  Lines         15066    15432     +366     
  Branches       1065     1093      +28     
============================================
+ Hits          10733    11031     +298     
- Misses         3866     3924      +58     
- Partials        467      477      +10     
Files Changed Coverage Δ
.../clients/jedis/gears/resps/FunctionStreamInfo.java 12.12% <12.12%> (ø)
...redis/clients/jedis/gears/TFunctionListParams.java 83.33% <83.33%> (ø)
...s/clients/jedis/gears/resps/StreamTriggerInfo.java 86.30% <86.30%> (ø)
...a/redis/clients/jedis/gears/resps/TriggerInfo.java 88.00% <88.00%> (ø)
...is/clients/jedis/gears/resps/GearsLibraryInfo.java 88.50% <88.50%> (ø)
.../redis/clients/jedis/gears/resps/FunctionInfo.java 91.30% <91.30%> (ø)
...redis/clients/jedis/gears/TFunctionLoadParams.java 91.66% <91.66%> (ø)
.../main/java/redis/clients/jedis/CommandObjects.java 85.77% <95.45%> (+0.03%) ⬆️
.../redis/clients/jedis/gears/RedisGearsProtocol.java 95.45% <95.45%> (ø)
.../main/java/redis/clients/jedis/BuilderFactory.java 76.05% <100.00%> (-0.06%) ⬇️
... and 4 more

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sazzad16 sazzad16 merged commit 1f418b0 into redis:master Sep 23, 2023
4 checks passed
@sazzad16 sazzad16 deleted the search-test-regression branch September 23, 2023 09:34
chenshi5012 pushed a commit to chenshi5012/jedis that referenced this pull request Oct 19, 2023
* 'master' of https://github.com/chenshi5012/jedis: (21 commits)
  Bump org.json:json from 20230618 to 20231013 (redis#3586)
  Fix SORTABLE argument issue in FT.CREATE command (redis#3584)
  Linking to Redis resources (redis#3583)
  Remove patch version snapshots
  Make integration and snapshot actions manually triggerable (redis#3579)
  Missing piece from redis#3578
  Release minor version snapshots (redis#3578)
  Allow getting schema field name (redis#3576)
  Fix binary variants of XRANGE and XREAD commands (redis#3571)
  Support GEOSHAPE field type in RediSearch (redis#3561)
  Bump org.apache.commons:commons-pool2 from 2.11.1 to 2.12.0 (redis#3565)
  Bump junixsocket-core to 2.8.1 (redis#3573)
  Bump org.apache.maven.plugins:maven-javadoc-plugin from 3.5.0 to 3.6.0 (redis#3539)
  Broadcast FUNCTION LOAD command methods (redis#3557)
  Encode map in encoded object (redis#3555)
  Polish Triggers and functions tests (redis#3554)
  Different variable names for json v1 and v2 interfaces (redis#3553)
  Fix Search/Gears test regression (redis#3552)
  Polish Feature: Triggers and functions commands (redis#3551)
  Feature: Triggers and functions commands (redis#3531)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants