Just return a plain stream

Author: jeffgbutler

src/main/java/org/mybatis/dynamic/sql/common/OrderByRenderer.java

@@ -22,7 +22,8 @@
 
 public class OrderByRenderer {
     public FragmentAndParameters render(OrderByModel orderByModel) {
-        String phrase = orderByModel.columns().map(this::calculateOrderByPhrase)
+        String phrase = orderByModel.columns()
+                .map(this::calculateOrderByPhrase)
                 .collect(Collectors.joining(", ", "order by ", "")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
         return FragmentAndParameters.fromFragment(phrase);
     }

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

@@ -32,7 +32,8 @@ private BatchInsertRenderer(Builder<T> builder) {
 
     public BatchInsert<T> render() {
         MultiRowValuePhraseVisitor visitor = new MultiRowValuePhraseVisitor(renderingStrategy, "row"); //$NON-NLS-1$)
-        FieldAndValueCollector collector = model.columnMappings().map(m -> m.accept(visitor))
+        FieldAndValueCollector collector = model.columnMappings()
+                .map(m -> m.accept(visitor))
                 .collect(FieldAndValueCollector.collect());
 
         String insertStatement = InsertRenderingUtilities.calculateInsertStatement(model.table(), collector);

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

@@ -38,7 +38,8 @@ private GeneralInsertRenderer(Builder builder) {
 
     public GeneralInsertStatementProvider render() {
         GeneralInsertValuePhraseVisitor visitor = new GeneralInsertValuePhraseVisitor(renderingContext);
-        FieldAndValueCollector collector = model.columnMappings().map(m -> m.accept(visitor))
+        FieldAndValueCollector collector = model.columnMappings()
+                .map(m -> m.accept(visitor))
                 .filter(Optional::isPresent)
                 .map(Optional::get)
                 .collect(FieldAndValueCollector.collect());

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

@@ -35,7 +35,8 @@ private InsertRenderer(Builder<T> builder) {
     public InsertStatementProvider<T> render() {
         ValuePhraseVisitor visitor = new ValuePhraseVisitor(renderingStrategy);
 
-        FieldAndValueCollector collector = model.columnMappings().map(m -> m.accept(visitor))
+        FieldAndValueCollector collector = model.columnMappings()
+                .map(m -> m.accept(visitor))
                 .filter(Optional::isPresent)
                 .map(Optional::get)
                 .collect(FieldAndValueCollector.collect());

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

@@ -63,7 +63,8 @@ private Optional<String> calculateColumnsPhrase() {
     }
 
     private String calculateColumnsPhrase(InsertColumnListModel columnList) {
-        return columnList.columns().map(SqlColumn::name)
+        return columnList.columns()
+                .map(SqlColumn::name)
                 .collect(Collectors.joining(", ", "(", ")")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
     }
 

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

@@ -36,7 +36,8 @@ public MultiRowInsertStatementProvider<T> render() {
         // the prefix is a generic format that will be resolved below with String.format(...)
         MultiRowValuePhraseVisitor visitor =
                 new MultiRowValuePhraseVisitor(renderingStrategy, "records[%s]"); //$NON-NLS-1$
-        FieldAndValueCollector collector = model.columnMappings().map(m -> m.accept(visitor))
+        FieldAndValueCollector collector = model.columnMappings()
+                .map(m -> m.accept(visitor))
                 .collect(FieldAndValueCollector.collect());
 
         String insertStatement = calculateInsertStatement(collector);

src/main/java/org/mybatis/dynamic/sql/select/render/JoinRenderer.java

@@ -39,7 +39,8 @@ private JoinRenderer(Builder builder) {
     }
 
     public FragmentAndParameters render() {
-        return joinModel.joinSpecifications().map(this::renderJoinSpecification)
+        return joinModel.joinSpecifications()
+                .map(this::renderJoinSpecification)
                 .collect(FragmentCollector.collect())
                 .toFragmentAndParameters(Collectors.joining(" ")); //$NON-NLS-1$
     }

src/main/java/org/mybatis/dynamic/sql/select/render/QueryExpressionRenderer.java

@@ -130,7 +130,8 @@ private FragmentAndParameters calculateQueryExpressionStart() {
     }
 
     private FragmentAndParameters calculateColumnList() {
-        return queryExpression.columns().map(this::renderColumnAndAlias)
+        return queryExpression.columns()
+                .map(this::renderColumnAndAlias)
                 .collect(FragmentCollector.collect())
                 .toFragmentAndParameters(Collectors.joining(", ")); //$NON-NLS-1$
     }
@@ -175,7 +176,8 @@ private Optional<FragmentAndParameters> calculateGroupByClause() {
     }
 
     private FragmentAndParameters renderGroupBy(GroupByModel groupByModel) {
-        return groupByModel.columns().map(this::renderColumn)
+        return groupByModel.columns()
+                .map(this::renderColumn)
                 .collect(FragmentCollector.collect())
                 .toFragmentAndParameters(
                         Collectors.joining(", ", "group by ", "")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$)

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

@@ -77,8 +77,8 @@ private FragmentAndParameters calculateUpdateStatementStart() {
     private FragmentAndParameters calculateSetPhrase() {
         SetPhraseVisitor visitor = new SetPhraseVisitor(renderingContext);
 
-        List<Optional<FragmentAndParameters>> fragmentsAndParameters =
-                updateModel.columnMappings().map(m -> m.accept(visitor))
+        List<Optional<FragmentAndParameters>> fragmentsAndParameters = updateModel.columnMappings()
+                        .map(m -> m.accept(visitor))
                         .collect(Collectors.toList());
 
         Validator.assertFalse(fragmentsAndParameters.stream().noneMatch(Optional::isPresent),

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

@@ -47,7 +47,8 @@ private DefaultConditionVisitor(Builder<T> builder) {
     @Override
     public FragmentAndParameters visit(AbstractListValueCondition<T> condition) {
         FragmentCollector fc = condition.values()
-                .map(this::toFragmentAndParameters).collect(FragmentCollector.collect());
+                .map(this::toFragmentAndParameters)
+                .collect(FragmentCollector.collect());
 
         String joinedFragments =
                 fc.collectFragments(Collectors.joining(",", "(", ")")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$