style: Cleaned up code

Author: Julien Ruaux

pom.xml

@@ -26,7 +26,7 @@
 		<github.owner>redis-field-engineering</github.owner>
 		<github.repo>redis-sql-trino</github.repo>
 
-		<lettucemod.version>3.2.1</lettucemod.version>
+		<lettucemod.version>3.3.1</lettucemod.version>
 		<lettuce.version>6.2.2.RELEASE</lettuce.version>
 		<testcontainers-redis.version>1.6.3</testcontainers-redis.version>
 		<ulid.version>5.1.0</ulid.version>

src/main/java/com/redis/trino/RediSearchPageSink.java

@@ -48,7 +48,6 @@
 import java.util.concurrent.CompletableFuture;
 
 import com.github.f4b6a3.ulid.UlidFactory;
-import com.google.common.collect.ImmutableList;
 import com.google.common.primitives.Shorts;
 import com.google.common.primitives.SignedBytes;
 import com.redis.lettucemod.api.StatefulRedisModulesConnection;
@@ -204,7 +203,7 @@ public static String value(Type type, Block block, int position) {
 
 	@Override
 	public CompletableFuture<Collection<Slice>> finish() {
-		return completedFuture(ImmutableList.of());
+		return completedFuture(List.of());
 	}
 
 	@Override

src/main/java/com/redis/trino/RediSearchTranslator.java

@@ -25,19 +25,12 @@
 
 import static java.util.Objects.requireNonNull;
 
-import java.util.Collections;
-import java.util.List;
-import java.util.Optional;
-
 import com.redis.lettucemod.search.AggregateOptions;
 import com.redis.lettucemod.search.CursorOptions;
 import com.redis.lettucemod.search.Limit;
 import com.redis.lettucemod.search.SearchOptions;
 import com.redis.lettucemod.search.SearchOptions.Builder;
 
-import io.lettuce.core.RedisURI;
-import io.trino.spi.HostAddress;
-
 public class RediSearchTranslator {
 
 	private final RediSearchQueryBuilder queryBuilder = new RediSearchQueryBuilder();
@@ -52,15 +45,6 @@ public RediSearchConfig getConfig() {
 		return config;
 	}
 
-	public List<HostAddress> getAddresses() {
-		Optional<String> uri = config.getUri();
-		if (uri.isPresent()) {
-			RedisURI redisURI = RedisURI.create(uri.get());
-			return Collections.singletonList(HostAddress.fromParts(redisURI.getHost(), redisURI.getPort()));
-		}
-		return Collections.emptyList();
-	}
-
 	public static class Aggregation {
 		private String index;
 		private String query;

src/test/java/com/redis/trino/RediSearchServer.java

@@ -12,8 +12,8 @@ public class RediSearchServer implements Closeable {
 
 	public RediSearchServer() {
 		this.dockerContainer = new RedisStackContainer(
-				RedisStackContainer.DEFAULT_IMAGE_NAME.withTag(RedisStackContainer.DEFAULT_TAG))
-				.withEnv("REDISEARCH_ARGS", "MAXAGGREGATERESULTS 1000000");
+				RedisStackContainer.DEFAULT_IMAGE_NAME.withTag(RedisStackContainer.DEFAULT_TAG));
+		this.dockerContainer.withEnv("REDISEARCH_ARGS", "MAXAGGREGATERESULTS -1");
 		this.dockerContainer.start();
 		this.context = new RedisTestContext(dockerContainer);
 	}