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.

Sign up for GitHub

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

Jump to bottom

Feature: Triggers and functions commands #3531

Merged
merged 3 commits into from
Sep 23, 2023
Merged

Feature: Triggers and functions commands #3531

merged 3 commits into from
Sep 23, 2023

Conversation

bsbodden
Copy link
Contributor

Implements Triggers and Functions Commands:

@bsbodden bsbodden force-pushed the bsb/feature-triggers-and-functions branch from 72a0473 to 56f57ad Compare September 12, 2023 16:16
@sazzad16 sazzad16 added this to the 5.1.0 milestone Sep 13, 2023
@bsbodden bsbodden force-pushed the bsb/feature-triggers-and-functions branch from 5e6c87c to b9de0e3 Compare September 15, 2023 01:02
sazzad16
sazzad16 requested changes Sep 17, 2023
View reviewed changes
Copy link
Collaborator

@sazzad16 sazzad16 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With default (RESP2):

Error:  Failures: 
Error:    GearsTest.testLibraryCallHashResult:433 expected:<7> but was:<14>

@bsbodden bsbodden force-pushed the bsb/feature-triggers-and-functions branch from 243890c to d0a4fe2 Compare September 18, 2023 14:42
@bsbodden
Copy link
Contributor Author

With default (RESP2):

Error:  Failures: 
Error:    GearsTest.testLibraryCallHashResult:433 expected:<7> but was:<14>

I was trying to check for the protocol in the tests assuming RedisProtocolUtil.getRedisProtocol() would return the right value. I changed to just check the data in flight for the test.

@bsbodden
Copy link
Contributor Author

@sazzad16 I only see a failure now that seems unrelated:

Error:  Failures: 
Error:    ClusterPipeliningTest.multiple:1072->assertThreadsCount:1098 
Expected: a value less than or equal to <20>
     but: <21> was greater than <20>
[INFO] 
Error:  Tests run: 1241, Failures: 1, Errors: 0, Skipped: 27

@sazzad16 sazzad16 merged commit eadf6a0 into redis:master Sep 23, 2023
3 of 4 checks passed
@sazzad16 sazzad16 mentioned this pull request Sep 23, 2023
Merged
sazzad16 added a commit that referenced this pull request Sep 23, 2023
@sazzad16
Polish Feature: Triggers and functions commands (#3551)
3c15285 
Polish #3531
This was referenced Sep 23, 2023
Closed
Merged
sazzad16 added a commit that referenced this pull request Sep 23, 2023
@sazzad16
Polish Triggers and functions tests (#3554)
eb6893a 
Polish #3531 tests
chenshi5012 pushed a commit to chenshi5012/jedis that referenced this pull request Oct 19, 2023
Merge branch 'master' of https://github.com/chenshi5012/jedis
912eb75 
* '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
Reviewers

@sazzad16 sazzad16 Awaiting requested review from sazzad16

Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
5.1.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@bsbodden @sazzad16