Use Collection instead of List in the Public API

src/main/java/org/mybatis/dynamic/sql/AbstractListValueCondition.java

@@ -1,5 +1,5 @@
 /**
- *    Copyright 2016-2018 the original author or authors.
+ *    Copyright 2016-2019 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -16,21 +16,21 @@
 package org.mybatis.dynamic.sql;
 
 import java.util.ArrayList;
-import java.util.List;
+import java.util.Collection;
 import java.util.Objects;
 import java.util.function.Function;
 import java.util.function.UnaryOperator;
 import java.util.stream.Stream;
 
 public abstract class AbstractListValueCondition<T> implements VisitableCondition<T> {
-    protected List<T> values;
+    protected Collection<T> values;
     protected UnaryOperator<Stream<T>> valueStreamOperations;
 
-    protected AbstractListValueCondition(List<T> values) {
+    protected AbstractListValueCondition(Collection<T> values) {
         this(values, UnaryOperator.identity());
     }
 
-    protected AbstractListValueCondition(List<T> values, UnaryOperator<Stream<T>> valueStreamOperations) {
+    protected AbstractListValueCondition(Collection<T> values, UnaryOperator<Stream<T>> valueStreamOperations) {
         this.values = new ArrayList<>(Objects.requireNonNull(values));
         this.valueStreamOperations = Objects.requireNonNull(valueStreamOperations);
     }

src/main/java/org/mybatis/dynamic/sql/SqlBuilder.java

@@ -1,5 +1,5 @@
 /**
- *    Copyright 2016-2018 the original author or authors.
+ *    Copyright 2016-2019 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -16,7 +16,7 @@
 package org.mybatis.dynamic.sql;
 
 import java.util.Arrays;
-import java.util.List;
+import java.util.Collection;
 import java.util.function.Supplier;
 
 import org.mybatis.dynamic.sql.delete.DeleteDSL;
@@ -113,7 +113,7 @@ static <T> BatchInsertDSL.IntoGatherer<T> insert(T...records) {
         return BatchInsertDSL.insert(records);
     }
 
-    static <T> BatchInsertDSL.IntoGatherer<T> insert(List<T> records) {
+    static <T> BatchInsertDSL.IntoGatherer<T> insert(Collection<T> records) {
         return BatchInsertDSL.insert(records);
     }
 
@@ -417,7 +417,7 @@ static <T> IsIn<T> isIn(T...values) {
         return isIn(Arrays.asList(values));
     }
 
-    static <T> IsIn<T> isIn(List<T> values) {
+    static <T> IsIn<T> isIn(Collection<T> values) {
         return IsIn.of(values);
     }
 
@@ -430,7 +430,7 @@ static <T> IsInWhenPresent<T> isInWhenPresent(T...values) {
         return isInWhenPresent(Arrays.asList(values));
     }
 
-    static <T> IsInWhenPresent<T> isInWhenPresent(List<T> values) {
+    static <T> IsInWhenPresent<T> isInWhenPresent(Collection<T> values) {
         return IsInWhenPresent.of(values);
     }
 
@@ -439,7 +439,7 @@ static <T> IsNotIn<T> isNotIn(T...values) {
         return isNotIn(Arrays.asList(values));
     }
 
-    static <T> IsNotIn<T> isNotIn(List<T> values) {
+    static <T> IsNotIn<T> isNotIn(Collection<T> values) {
         return IsNotIn.of(values);
     }
 
@@ -452,7 +452,7 @@ static <T> IsNotInWhenPresent<T> isNotInWhenPresent(T...values) {
         return isNotInWhenPresent(Arrays.asList(values));
     }
 
-    static <T> IsNotInWhenPresent<T> isNotInWhenPresent(List<T> values) {
+    static <T> IsNotInWhenPresent<T> isNotInWhenPresent(Collection<T> values) {
         return IsNotInWhenPresent.of(values);
     }
 
@@ -558,31 +558,31 @@ static IsInCaseInsensitive isInCaseInsensitive(String...values) {
         return isInCaseInsensitive(Arrays.asList(values));
     }
 
-    static IsInCaseInsensitive isInCaseInsensitive(List<String> values) {
+    static IsInCaseInsensitive isInCaseInsensitive(Collection<String> values) {
         return IsInCaseInsensitive.of(values);
     }
 
     static IsInCaseInsensitiveWhenPresent isInCaseInsensitiveWhenPresent(String...values) {
         return isInCaseInsensitiveWhenPresent(Arrays.asList(values));
     }
 
-    static IsInCaseInsensitiveWhenPresent isInCaseInsensitiveWhenPresent(List<String> values) {
+    static IsInCaseInsensitiveWhenPresent isInCaseInsensitiveWhenPresent(Collection<String> values) {
         return IsInCaseInsensitiveWhenPresent.of(values);
     }
 
     static IsNotInCaseInsensitive isNotInCaseInsensitive(String...values) {
         return isNotInCaseInsensitive(Arrays.asList(values));
     }
 
-    static IsNotInCaseInsensitive isNotInCaseInsensitive(List<String> values) {
+    static IsNotInCaseInsensitive isNotInCaseInsensitive(Collection<String> values) {
         return IsNotInCaseInsensitive.of(values);
     }
 
     static IsNotInCaseInsensitiveWhenPresent isNotInCaseInsensitiveWhenPresent(String...values) {
         return isNotInCaseInsensitiveWhenPresent(Arrays.asList(values));
     }
 
-    static IsNotInCaseInsensitiveWhenPresent isNotInCaseInsensitiveWhenPresent(List<String> values) {
+    static IsNotInCaseInsensitiveWhenPresent isNotInCaseInsensitiveWhenPresent(Collection<String> values) {
         return IsNotInCaseInsensitiveWhenPresent.of(values);
     }
 

src/main/java/org/mybatis/dynamic/sql/insert/BatchInsertDSL.java

@@ -1,5 +1,5 @@
 /**
- *    Copyright 2016-2017 the original author or authors.
+ *    Copyright 2016-2019 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -17,6 +17,7 @@
 
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collection;
 import java.util.List;
 
 import org.mybatis.dynamic.sql.SqlColumn;
@@ -29,11 +30,11 @@
 
 public class BatchInsertDSL<T> {
 
-    private List<T> records;
+    private Collection<T> records;
     private SqlTable table;
     private List<InsertMapping> columnMappings = new ArrayList<>();
 
-    private BatchInsertDSL(List<T> records, SqlTable table) {
+    private BatchInsertDSL(Collection<T> records, SqlTable table) {
         this.records = records;
         this.table = table;
     }
@@ -54,14 +55,14 @@ public static <T> IntoGatherer<T> insert(T...records) {
         return new IntoGatherer<>(Arrays.asList(records));
     }
 
-    public static <T> IntoGatherer<T> insert(List<T> records) {
+    public static <T> IntoGatherer<T> insert(Collection<T> records) {
         return new IntoGatherer<>(records);
     }
 
     public static class IntoGatherer<T> {
-        private List<T> records;
+        private Collection<T> records;
 
-        private IntoGatherer(List<T> records) {
+        private IntoGatherer(Collection<T> records) {
             this.records = records;
         }
 

src/main/java/org/mybatis/dynamic/sql/insert/BatchInsertModel.java

@@ -1,5 +1,5 @@
 /**
- *    Copyright 2016-2017 the original author or authors.
+ *    Copyright 2016-2019 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -16,6 +16,7 @@
 package org.mybatis.dynamic.sql.insert;
 
 import java.util.ArrayList;
+import java.util.Collection;
 import java.util.Collections;
 import java.util.List;
 import java.util.Objects;
@@ -58,7 +59,7 @@ public BatchInsert<T> render(RenderingStrategy renderingStrategy) {
                 .render();
     }
 
-    public static <T> Builder<T> withRecords(List<T> records) {
+    public static <T> Builder<T> withRecords(Collection<T> records) {
         return new Builder<T>().withRecords(records);
     }
 
@@ -72,7 +73,7 @@ public Builder<T> withTable(SqlTable table) {
             return this;
         }
 
-        public Builder<T> withRecords(List<T> records) {
+        public Builder<T> withRecords(Collection<T> records) {
             this.records.addAll(records);
             return this;
         }

src/main/java/org/mybatis/dynamic/sql/where/condition/IsIn.java

@@ -1,5 +1,5 @@
 /**
- *    Copyright 2016-2018 the original author or authors.
+ *    Copyright 2016-2019 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -17,7 +17,7 @@
 
 import static org.mybatis.dynamic.sql.util.StringUtilities.spaceAfter;
 
-import java.util.List;
+import java.util.Collection;
 import java.util.function.UnaryOperator;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
@@ -26,11 +26,11 @@
 
 public class IsIn<T> extends AbstractListValueCondition<T> {
 
-    protected IsIn(List<T> values, UnaryOperator<Stream<T>> valueStreamOperations) {
+    protected IsIn(Collection<T> values, UnaryOperator<Stream<T>> valueStreamOperations) {
         super(values, valueStreamOperations);
     }
 
-    protected IsIn(List<T> values) {
+    protected IsIn(Collection<T> values) {
         super(values);
     }
 
@@ -44,7 +44,7 @@ public IsIn<T> withValueStreamOperations(UnaryOperator<Stream<T>> valueStreamOpe
         return new IsIn<>(values, valueStreamOperations);
     }
 
-    public static <T> IsIn<T> of(List<T> values) {
+    public static <T> IsIn<T> of(Collection<T> values) {
         return new IsIn<>(values);
     }
 }

src/main/java/org/mybatis/dynamic/sql/where/condition/IsInCaseInsensitive.java

@@ -1,5 +1,5 @@
 /**
- *    Copyright 2016-2018 the original author or authors.
+ *    Copyright 2016-2019 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -15,7 +15,7 @@
  */
 package org.mybatis.dynamic.sql.where.condition;
 
-import java.util.List;
+import java.util.Collection;
 import java.util.function.UnaryOperator;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
@@ -25,11 +25,11 @@
 
 public class IsInCaseInsensitive extends AbstractListValueCondition<String> {
 
-    protected IsInCaseInsensitive(List<String> values) {
+    protected IsInCaseInsensitive(Collection<String> values) {
         super(values, s -> s.map(StringUtilities::safelyUpperCase));
     }
 
-    protected IsInCaseInsensitive(List<String> values, UnaryOperator<Stream<String>> valueStreamOperations) {
+    protected IsInCaseInsensitive(Collection<String> values, UnaryOperator<Stream<String>> valueStreamOperations) {
         super(values, StringUtilities.upperCaseAfter(valueStreamOperations));
     }
 
@@ -43,7 +43,7 @@ public IsInCaseInsensitive withValueStreamOperations(UnaryOperator<Stream<String
         return new IsInCaseInsensitive(values, valueStreamOperations);
     }
 
-    public static IsInCaseInsensitive of(List<String> values) {
+    public static IsInCaseInsensitive of(Collection<String> values) {
         return new IsInCaseInsensitive(values);
     }
 }

src/main/java/org/mybatis/dynamic/sql/where/condition/IsInCaseInsensitiveWhenPresent.java

@@ -1,5 +1,5 @@
 /**
- *    Copyright 2016-2018 the original author or authors.
+ *    Copyright 2016-2019 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -15,16 +15,16 @@
  */
 package org.mybatis.dynamic.sql.where.condition;
 
-import java.util.List;
+import java.util.Collection;
 import java.util.Objects;
 
 public class IsInCaseInsensitiveWhenPresent extends IsInCaseInsensitive {
 
-    protected IsInCaseInsensitiveWhenPresent(List<String> values) {
+    protected IsInCaseInsensitiveWhenPresent(Collection<String> values) {
         super(values, s -> s.filter(Objects::nonNull));
     }
 
-    public static IsInCaseInsensitiveWhenPresent of(List<String> values) {
+    public static IsInCaseInsensitiveWhenPresent of(Collection<String> values) {
         return new IsInCaseInsensitiveWhenPresent(values);
     }
 }

src/main/java/org/mybatis/dynamic/sql/where/condition/IsInWhenPresent.java

@@ -1,5 +1,5 @@
 /**
- *    Copyright 2016-2018 the original author or authors.
+ *    Copyright 2016-2019 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -15,16 +15,16 @@
  */
 package org.mybatis.dynamic.sql.where.condition;
 
-import java.util.List;
+import java.util.Collection;
 import java.util.Objects;
 
 public class IsInWhenPresent<T> extends IsIn<T> {
 
-    protected IsInWhenPresent(List<T> values) {
+    protected IsInWhenPresent(Collection<T> values) {
         super(values, s -> s.filter(Objects::nonNull));
     }
 
-    public static <T> IsInWhenPresent<T> of(List<T> values) {
+    public static <T> IsInWhenPresent<T> of(Collection<T> values) {
         return new IsInWhenPresent<>(values);
     }
 }

src/main/java/org/mybatis/dynamic/sql/where/condition/IsNotIn.java

@@ -1,5 +1,5 @@
 /**
- *    Copyright 2016-2018 the original author or authors.
+ *    Copyright 2016-2019 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -17,7 +17,7 @@
 
 import static org.mybatis.dynamic.sql.util.StringUtilities.spaceAfter;
 
-import java.util.List;
+import java.util.Collection;
 import java.util.function.UnaryOperator;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
@@ -26,11 +26,11 @@
 
 public class IsNotIn<T> extends AbstractListValueCondition<T> {
 
-    protected IsNotIn(List<T> values, UnaryOperator<Stream<T>> valueStreamOperations) {
+    protected IsNotIn(Collection<T> values, UnaryOperator<Stream<T>> valueStreamOperations) {
         super(values, valueStreamOperations);
     }
 
-    protected IsNotIn(List<T> values) {
+    protected IsNotIn(Collection<T> values) {
         super(values);
     }
 
@@ -45,7 +45,7 @@ public IsNotIn<T> withValueStreamOperations(UnaryOperator<Stream<T>> valueStream
         return new IsNotIn<>(values, valueStreamOperations);
     }
 
-    public static <T> IsNotIn<T> of(List<T> values) {
+    public static <T> IsNotIn<T> of(Collection<T> values) {
         return new IsNotIn<>(values);
     }
 }