Use Collection instead of List as arguments type in TarantoolClient

.gitignore

@@ -1,5 +1,8 @@
 target/
 .idea/
+.settings/
+.classpath
+.project
 .rocks/
 tmp/
 *.iml

CHANGELOG.md

@@ -8,6 +8,7 @@
 
 ### Internal and API changes
 
+- **[breaking change]** replace `List` type with `Collection` for the `arguments` client API method parameters
 - Bump testcontainers-java-tarantool to 1.0.1 ([#400](https://github.com/tarantool/cartridge-java/issues/400))
 - Add `"mode"` option for select operation ([#107](https://github.com/tarantool/cartridge-java/issues/107))
 - Change using of proxy client parameters (`mode`, `rollback_on_error`, `stop_on_error`) with enum classes ([#419](https://github.com/tarantool/cartridge-java/issues/419))

src/main/java/io/tarantool/driver/api/MessagePackMapperBuilder.java

@@ -35,6 +35,13 @@ public interface MessagePackMapperBuilder {
      */
     MessagePackMapperBuilder withDefaultArrayValueConverter();
 
+    /**
+     * Configure the mapper with default {@link Collection} to {@code MP_ARRAY} entity converter
+     *
+     * @return builder
+     */
+    MessagePackMapperBuilder withDefaultCollectionObjectConverter();
+
     /**
      * Configure the mapper with default {@link List} to {@code MP_ARRAY} entity converter
      *

src/main/java/io/tarantool/driver/api/TarantoolCallOperations.java

@@ -8,6 +8,7 @@
 import io.tarantool.driver.mappers.factories.ResultMapperFactoryFactory;
 import org.msgpack.value.Value;
 
+import java.util.Collection;
 import java.util.List;
 import java.util.concurrent.CompletableFuture;
 import java.util.function.Supplier;
@@ -51,7 +52,7 @@ public interface TarantoolCallOperations {
      * @return some result
      * @throws TarantoolClientException if the client is not connected or some other error occurred
      */
-    CompletableFuture<List<?>> call(String functionName, List<?> arguments) throws TarantoolClientException;
+    CompletableFuture<List<?>> call(String functionName, Collection<?> arguments) throws TarantoolClientException;
 
     /**
      * Execute a function defined on Tarantool instance
@@ -62,7 +63,7 @@ public interface TarantoolCallOperations {
      * @return some result
      * @throws TarantoolClientException if the client is not connected
      */
-    CompletableFuture<List<?>> call(String functionName, List<?> arguments, MessagePackMapper mapper)
+    CompletableFuture<List<?>> call(String functionName, Collection<?> arguments, MessagePackMapper mapper)
         throws TarantoolClientException;
 
     /**
@@ -108,7 +109,7 @@ <T> CompletableFuture<T> call(
      */
     <T> CompletableFuture<TarantoolResult<T>> callForTupleResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         Class<T> entityClass)
         throws TarantoolClientException;
 
@@ -125,7 +126,7 @@ <T> CompletableFuture<TarantoolResult<T>> callForTupleResult(
      */
     <T> CompletableFuture<T> call(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         CallResultMapper<T, SingleValueCallResult<T>> resultMapper)
         throws TarantoolClientException;
 
@@ -144,7 +145,7 @@ <T> CompletableFuture<T> call(
      */
     <T> CompletableFuture<TarantoolResult<T>> callForTupleResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         MessagePackObjectMapper argumentsMapper,
         Class<T> entityClass)
         throws TarantoolClientException;
@@ -162,7 +163,7 @@ <T> CompletableFuture<TarantoolResult<T>> callForTupleResult(
      */
     <T> CompletableFuture<T> call(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         MessagePackObjectMapper argumentsMapper,
         CallResultMapper<T, SingleValueCallResult<T>> resultMapper)
         throws TarantoolClientException;
@@ -181,7 +182,7 @@ <T> CompletableFuture<T> call(
      */
     <T> CompletableFuture<T> callForSingleResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         MessagePackObjectMapper argumentsMapper,
         Class<T> resultClass)
         throws TarantoolClientException;
@@ -200,7 +201,7 @@ <T> CompletableFuture<T> callForSingleResult(
      */
     <T> CompletableFuture<T> callForSingleResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         MessagePackObjectMapper argumentsMapper,
         ValueConverter<Value, T> valueConverter)
         throws TarantoolClientException;
@@ -219,7 +220,7 @@ <T> CompletableFuture<T> callForSingleResult(
      */
     <T> CompletableFuture<T> callForSingleResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         MessagePackObjectMapper argumentsMapper,
         CallResultMapper<T, SingleValueCallResult<T>> resultMapper)
         throws TarantoolClientException;
@@ -237,7 +238,7 @@ <T> CompletableFuture<T> callForSingleResult(
      */
     <T> CompletableFuture<T> callForSingleResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         Class<T> resultClass)
         throws TarantoolClientException;
 
@@ -254,7 +255,7 @@ <T> CompletableFuture<T> callForSingleResult(
      */
     <T> CompletableFuture<T> callForSingleResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         ValueConverter<Value, T> valueConverter)
         throws TarantoolClientException;
 
@@ -271,7 +272,7 @@ <T> CompletableFuture<T> callForSingleResult(
      */
     <T> CompletableFuture<T> callForSingleResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         CallResultMapper<T, SingleValueCallResult<T>> resultMapper)
         throws TarantoolClientException;
 
@@ -335,7 +336,7 @@ <T> CompletableFuture<T> callForSingleResult(
      */
     <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         MessagePackObjectMapper argumentsMapper,
         Supplier<R> resultContainerSupplier,
         Class<T> resultClass)
@@ -356,7 +357,7 @@ <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
      */
     <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         MessagePackObjectMapper argumentsMapper,
         Supplier<R> resultContainerSupplier,
         ValueConverter<Value, T> valueConverter)
@@ -376,7 +377,7 @@ <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
      */
     <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         MessagePackObjectMapper argumentsMapper,
         CallResultMapper<R, MultiValueCallResult<T, R>> resultMapper)
         throws TarantoolClientException;
@@ -395,7 +396,7 @@ <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
      */
     <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         Supplier<R> resultContainerSupplier,
         Class<T> resultClass)
         throws TarantoolClientException;
@@ -414,7 +415,7 @@ <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
      */
     <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         Supplier<R> resultContainerSupplier,
         ValueConverter<Value, T> valueConverter)
         throws TarantoolClientException;
@@ -432,7 +433,7 @@ <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
      */
     <T, R extends List<T>> CompletableFuture<R> callForMultiResult(
         String functionName,
-        List<?> arguments,
+        Collection<?> arguments,
         CallResultMapper<R, MultiValueCallResult<T, R>> resultMapper)
         throws TarantoolClientException;
 

src/main/java/io/tarantool/driver/api/TarantoolEvalOperations.java

@@ -4,6 +4,7 @@
 import io.tarantool.driver.mappers.MessagePackObjectMapper;
 import io.tarantool.driver.mappers.MessagePackValueMapper;
 
+import java.util.Collection;
 import java.util.List;
 import java.util.concurrent.CompletableFuture;
 
@@ -35,7 +36,7 @@ public interface TarantoolEvalOperations {
      * @return some result
      * @throws TarantoolClientException if the client is not connected
      */
-    CompletableFuture<List<?>> eval(String expression, List<?> arguments) throws TarantoolClientException;
+    CompletableFuture<List<?>> eval(String expression, Collection<?> arguments) throws TarantoolClientException;
 
     /**
      * Execute a Lua expression in the Tarantool instance. If a result is expected, the expression must start with
@@ -60,7 +61,7 @@ CompletableFuture<List<?>> eval(String expression, MessagePackValueMapper result
      * @return some result
      * @throws TarantoolClientException if the client is not connected
      */
-    CompletableFuture<List<?>> eval(String expression, List<?> arguments, MessagePackValueMapper resultMapper)
+    CompletableFuture<List<?>> eval(String expression, Collection<?> arguments, MessagePackValueMapper resultMapper)
         throws TarantoolClientException;
 
     /**
@@ -76,7 +77,7 @@ CompletableFuture<List<?>> eval(String expression, List<?> arguments, MessagePac
      */
     CompletableFuture<List<?>> eval(
         String expression,
-        List<?> arguments,
+        Collection<?> arguments,
         MessagePackObjectMapper argumentsMapper,
         MessagePackValueMapper resultMapper) throws TarantoolClientException;
 }