Skip to content

feat: Added string options to search and aggregate commands - #74

Merged
jruaux merged 1 commit into
mainfrom
search-string-options
Jul 18, 2025
Merged

feat: Added string options to search and aggregate commands#74
jruaux merged 1 commit into
mainfrom
search-string-options

Conversation

@jruaux

@jruaux jruaux commented Jul 18, 2025

Copy link
Copy Markdown
Contributor

Summary

  • Enhanced FT.SEARCH and FT.AGGREGATE commands to accept string-based options alongside existing object-based options
  • Added string parameter overloads to search and aggregate methods across sync, async, and reactive APIs
  • Updated command builders to support both option types for improved flexibility

Changes

  • Core APIs: Added string option variants to RediSearchCommands, RediSearchAsyncCommands, and RediSearchReactiveCommands
  • Implementations: Updated command implementations in RedisModulesAsyncCommandsImpl, RedisModulesReactiveCommandsImpl, and cluster variants
  • Command Builder: Enhanced SearchCommandBuilder to handle string-based options
  • Tests: Added test coverage for new string option functionality

Test plan

  • Verify existing tests pass with new string option support
  • Add tests for string-based search and aggregate operations
  • Validate both object and string options work correctly
  • Test across sync, async, and reactive command variants

@jruaux
jruaux merged commit c5457af into main Jul 18, 2025
1 check passed
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.

1 participant