Merge pull request #824 from jeffgbutler/java-11-update

Update to Java 17

Author: jeffgbutler

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 This log will detail notable changes to MyBatis Dynamic SQL. Full details are available on the GitHub milestone pages.
 
+## Release 2.0.0 - Unreleased
+
+The library now requires Java 17.
+
 ## Release 1.5.2 - June 3, 2024
 
 This is a small maintenance release with the following changes:

README.md

@@ -80,4 +80,4 @@ The library test cases provide several complete examples of using the library in
 
 ## Requirements
 
-The library has no dependencies.  Java 8 or higher is required.
+The library has no dependencies.  Version 2.x requires Java 17. Version 1.8 requires Java 8.

pom.xml

@@ -26,7 +26,7 @@
 
   <groupId>org.mybatis.dynamic-sql</groupId>
   <artifactId>mybatis-dynamic-sql</artifactId>
-  <version>1.6.0-SNAPSHOT</version>
+  <version>2.0.0-SNAPSHOT</version>
 
   <name>MyBatis Dynamic SQL</name>
   <description>MyBatis framework for generating dynamic SQL</description>
@@ -57,8 +57,8 @@
 
   <properties>
     <java.version>17</java.version>
-    <java.release.version>8</java.release.version>
-    <junit.jupiter.version>5.10.3</junit.jupiter.version>
+    <java.release.version>17</java.release.version>
+    <junit.jupiter.version>5.10.2</junit.jupiter.version>
     <spring.batch.version>5.1.2</spring.batch.version>
 
     <checkstyle.config>checkstyle-override.xml</checkstyle.config>

src/main/java/org/mybatis/dynamic/sql/insert/render/GeneralInsertRenderer.java

@@ -40,8 +40,7 @@ private GeneralInsertRenderer(Builder builder) {
     public GeneralInsertStatementProvider render() {
         FieldAndValueCollector collector = model.columnMappings()
                 .map(m -> m.accept(visitor))
-                .filter(Optional::isPresent)
-                .map(Optional::get)
+                .flatMap(Optional::stream)
                 .collect(FieldAndValueCollector.collect());
 
         Validator.assertFalse(collector.isEmpty(), "ERROR.9"); //$NON-NLS-1$

src/main/java/org/mybatis/dynamic/sql/insert/render/InsertRenderer.java

@@ -35,8 +35,7 @@ private InsertRenderer(Builder<T> builder) {
     public InsertStatementProvider<T> render() {
         FieldAndValueCollector collector = model.columnMappings()
                 .map(m -> m.accept(visitor))
-                .filter(Optional::isPresent)
-                .map(Optional::get)
+                .flatMap(Optional::stream)
                 .collect(FieldAndValueCollector.collect());
 
         Validator.assertFalse(collector.isEmpty(), "ERROR.10"); //$NON-NLS-1$

src/main/java/org/mybatis/dynamic/sql/update/render/UpdateRenderer.java

@@ -85,8 +85,7 @@ private FragmentAndParameters calculateSetPhrase() {
                 "ERROR.18"); //$NON-NLS-1$
 
         FragmentCollector fragmentCollector = fragmentsAndParameters.stream()
-                .filter(Optional::isPresent)
-                .map(Optional::get)
+                .flatMap(Optional::stream)
                 .collect(FragmentCollector.collect());
 
         return toSetPhrase(fragmentCollector);

src/main/java/org/mybatis/dynamic/sql/where/render/CriterionRenderer.java

@@ -135,8 +135,7 @@ private FragmentAndParameters renderExists(ExistsCriterion criterion) {
 
     private List<RenderedCriterion> renderSubCriteria(List<AndOrCriteriaGroup> subCriteria) {
         return subCriteria.stream().map(this::renderAndOrCriteriaGroup)
-                .filter(Optional::isPresent)
-                .map(Optional::get)
+                .flatMap(Optional::stream)
                 .collect(Collectors.toList());
     }
 

src/site/markdown/docs/conditions.md

@@ -267,7 +267,7 @@ any null or blank string, and you want to trim all strings. This can be accompli
             .where(animalName, isIn("  Mouse", "  ", null, "", "Musk shrew  ")
                     .filter(Objects::nonNull)
                     .map(String::trim)
-                    .filter(st -> !st.isEmpty()))
+                    .filter(not(String::isEmpty)))
             .orderBy(id)
             .build()
             .render(RenderingStrategies.MYBATIS3);
@@ -284,7 +284,7 @@ public class MyInCondition {
         return SqlBuilder.isIn(values)
                .filter(Objects::nonNull)
                .map(String::trim)
-               .filter(st -> !st.isEmpty());
+               .filter(not(String::isEmpty));
     }
 }
 ```

src/test/java/examples/animal/data/MyInCondition.java

@@ -15,6 +15,8 @@
  */
 package examples.animal.data;
 
+import static java.util.function.Predicate.not;
+
 import java.util.Objects;
 
 import org.mybatis.dynamic.sql.SqlBuilder;
@@ -25,6 +27,6 @@ public static IsIn<String> isIn(String...values) {
         return SqlBuilder.isIn(values)
                 .filter(Objects::nonNull)
                 .map((String::trim))
-                .filter(st -> !st.isEmpty());
+                .filter(not(String::isEmpty));
     }
 }

src/test/java/examples/animal/data/OptionalConditionsWithPredicatesAnimalDataTest.java

@@ -20,6 +20,7 @@
 import static examples.animal.data.AnimalDataDynamicSqlSupport.bodyWeight;
 import static examples.animal.data.AnimalDataDynamicSqlSupport.brainWeight;
 import static examples.animal.data.AnimalDataDynamicSqlSupport.id;
+import static java.util.function.Predicate.not;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.junit.jupiter.api.Assertions.assertAll;
 import static org.mybatis.dynamic.sql.SqlBuilder.*;
@@ -484,7 +485,7 @@ void testValueStreamTransformer() {
                     .where(animalName, isIn("  Mouse", "  ", null, "", "Musk shrew  ")
                             .filter(Objects::nonNull)
                                     .map(String::trim)
-                                    .filter(st -> !st.isEmpty()))
+                                    .filter(not(String::isEmpty)))
                     .orderBy(id)
                     .build()
                     .render(RenderingStrategies.MYBATIS3);