Align Search APIs and behaviour with Redis 8.0+

[uglide]

• Introduce executeKeylessCommand
• Implement round-robin execution in ClusterCommandExecutor
• Update search commands according to the latest request/response policies
• Remove JedisBroadcastAndRoundRobinConfig
• Introduce AggregateIterator for proper Cluster support
• Deprecate APIs that rely on legacy RediSeach behaviour

[github-actions]

Test Results

  249 files  + 1    249 suites  +1   9m 26s ⏱️ -2s
4 614 tests +18  4 610 ✅ +61  4 💤  - 43  0 ❌ ±0 
2 483 runs  + 9  2 483 ✅ + 9  0 💤 ± 0  0 ❌ ±0 

Results for commit 270a625. ± Comparison against base commit 60c356b.

♻️ This comment has been updated with latest results.

[Copilot]

Pull Request Overview

This PR aligns various Redis search APIs with Redis 8.0+ policies by introducing a new keyless command execution method, implementing round‑robin distribution for keyless commands in cluster mode, and replacing legacy search iteration APIs with a more robust AggregateIterator. Key changes include the introduction of executeKeylessCommand, removal of JedisBroadcastAndRoundRobinConfig, and the implementation of round‑robin scheduling in ClusterCommandExecutor.

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/test/java/redis/clients/jedis/modules/search/AggregateIteratorTest.java	New tests for AggregateIterator covering basic use, cursor removal and error cases
src/main/java/redis/clients/jedis/search/aggr/AggregationBuilder.java	Adds cursorCount tracking when using cursor-based aggregation
src/main/java/redis/clients/jedis/search/aggr/AggregateIterator.java	Implements a new iterator for aggregation results with proper cursor management
src/main/java/redis/clients/jedis/executors/ClusterCommandExecutor.java	Introduces round‑robin distribution for keyless command execution
src/main/java/redis/clients/jedis/UnifiedJedis.java	Updates search commands to use the new executeKeylessCommand method and deprecates legacy APIs
src/main/java/redis/clients/jedis/CommandObjects.java	Adjusts round‑robin command routing logic and removes unused configuration

Comments suppressed due to low confidence (1)

src/main/java/redis/clients/jedis/CommandObjects.java:3443

• [nitpick] Add a comment explaining why certain SearchCommands (SUGGET, SUGADD, SUGLEN, SUGDEL, CURSOR) are excluded from round-robin routing to aid future maintainers.

private boolean isRoundRobinSearchCommand(SearchCommand sc) {

[Copilot]

Consider caching the node list or optimizing the round-robin selection process if the connection map does not change frequently, to reduce repetitive list creation and improve efficiency.

Suggested change

	// Convert connection map to list for round-robin access
	List<Map.Entry<String, ConnectionPool>> nodeList = new ArrayList<>(connectionMap.entrySet());
	// Select node using round-robin distribution for true unified distribution
	// Use modulo directly on the node list size to create a circular counter
	int roundRobinIndex = roundRobinCounter.getAndUpdate(current -> (current + 1) % nodeList.size());
	Map.Entry<String, ConnectionPool> selectedEntry = nodeList.get(roundRobinIndex);
	// Check if the connection map size has changed
	if (cachedNodeList == null || cachedConnectionMapSize != connectionMap.size()) {
	synchronized (this) {
	if (cachedNodeList == null || cachedConnectionMapSize != connectionMap.size()) {
	cachedNodeList = new ArrayList<>(connectionMap.entrySet());
	cachedConnectionMapSize = connectionMap.size();
	}
	}
	}
	// Select node using round-robin distribution for true unified distribution
	// Use modulo directly on the node list size to create a circular counter
	int roundRobinIndex = roundRobinCounter.getAndUpdate(current -> (current + 1) % cachedNodeList.size());
	Map.Entry<String, ConnectionPool> selectedEntry = cachedNodeList.get(roundRobinIndex);