Skip to content

[Redis 8.10] Add FT.ALIASLIST search command support - #4619

Merged
ggivo merged 7 commits into
masterfrom
im/8_10-ft-alias-list
Jul 29, 2026
Merged

ggivo merged 7 commits into
masterfrom
im/8_10-ft-alias-list

Conversation

@uglide

@uglide uglide commented Jul 14, 2026 •

Copy link
Copy Markdown
Contributor

Add support for the FT.ALIASLIST command (Query Engine 8.10), which lists all aliases associated with a given index.

  • Add ALIASLIST to the SearchCommand protocol enum
  • Add CommandObjects.ftAliasList(indexName) using STRING_SET, routed as a keyless round-robin search command
  • Declare ftAliasList in RediSearchCommands (Set) and RediSearchPipelineCommands (Response<Set>)
  • Implement in UnifiedJedis and PipeliningBase
  • Add integration test covering the empty-index, multiple-aliases, and non-existent-index (error propagated) cases

Note

Low Risk
Read-only search command added in parallel with existing FT alias APIs; no auth, data mutation, or cluster routing changes beyond the new command.

Overview
Adds client support for FT.ALIASLIST, which returns all RediSearch aliases for a given index as a Set<String>.

The change wires the command through the usual stack: SearchCommand.ALIASLIST in the protocol enum, CommandObjects.ftAliasList (round-robin search routing, STRING_SET builder), and matching methods on RediSearchCommands, RediSearchPipelineCommands, UnifiedJedis, and PipeliningBase.

Integration coverage (aliasList, gated with @SinceRedisVersion for 8.10) checks empty alias sets, multiple aliases after add/delete, and server errors for a missing index.

Reviewed by Cursor Bugbot for commit d165c3d. Bugbot is set up for automated code reviews on this repo. Configure here.

@uglide uglide changed the title Add FT.ALIASLIST search command support [Redis 8.10] Add FT.ALIASLIST search command support Jul 14, 2026
@github-actions

github-actions Bot commented Jul 14, 2026 •

Copy link
Copy Markdown

Test Results

  206 files  ±0    206 suites  ±0   9m 35s ⏱️ -11s
8 044 tests +4  7 984 ✅ +197  60 💤  - 193  0 ❌ ±0 
8 064 runs  +4  8 004 ✅ +197  60 💤  - 193  0 ❌ ±0 

Results for commit d165c3d. ± Comparison against base commit 1aa8f07.

♻️ This comment has been updated with latest results.

uglide added 2 commits July 14, 2026 12:38
Add support for the FT.ALIASLIST command (Query Engine 8.10), which
lists all aliases associated with a given index.

- Add ALIASLIST to the SearchCommand protocol enum
- Add CommandObjects.ftAliasList(indexName) using STRING_SET, routed as
  a keyless round-robin search command
- Declare ftAliasList in RediSearchCommands (Set<String>) and
  RediSearchPipelineCommands (Response<Set<String>>)
- Implement in UnifiedJedis and PipeliningBase
- Add integration test covering the empty-index, multiple-aliases, and
  non-existent-index (error propagated) cases
@uglide
uglide force-pushed the im/8_10-ft-alias-list branch from 91cec46 to 86ae7a7 Compare July 14, 2026 10:38
@ggivo ggivo added this to the 8.0.0 milestone Jul 28, 2026
@ggivo ggivo added the feature label Jul 28, 2026
@uglide
uglide requested a review from ggivo July 28, 2026 15:36
ggivo added 2 commits July 29, 2026 09:20
…sion gate

The 8.10-rc2 image reports redis_version 8.9.241, so gating at 8.10.0
silently skipped the test in CI even though the command is present;
the named constant carries the rc mapping.

@ggivo ggivo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@ggivo
ggivo merged commit 365a077 into master Jul 29, 2026
15 checks passed
@ggivo
ggivo deleted the im/8_10-ft-alias-list branch July 31, 2026 09:50
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.

2 participants