Merge branch 'master' into gh-100

Conflicts:
	src/main/java/org/mybatis/dynamic/sql/select/QueryExpressionDSL.java
	src/main/java/org/mybatis/dynamic/sql/select/SelectDSL.java

Author: jeffgbutler

src/main/java/org/mybatis/dynamic/sql/select/QueryExpressionDSL.java

@@ -53,15 +53,34 @@ public class QueryExpressionDSL<R> implements Buildable<R> {
     private Supplier<R> buildDelegateMethod;
 
     private QueryExpressionDSL(FromGatherer<R> fromGatherer) {
-        connector = fromGatherer.builder.connector;
-        selectList = Arrays.asList(fromGatherer.builder.selectList);
-        isDistinct = fromGatherer.builder.isDistinct;
-        selectDSL = Objects.requireNonNull(fromGatherer.builder.selectDSL);
+        connector = fromGatherer.connector;
+        selectList = Arrays.asList(fromGatherer.selectList);
+        isDistinct = fromGatherer.isDistinct;
+        selectDSL = Objects.requireNonNull(fromGatherer.selectDSL);
         table = Objects.requireNonNull(fromGatherer.table);
-        tableAliases.putAll(fromGatherer.tableAliasMap);
         buildDelegateMethod = this::internalBuild;
     }
 
+    private QueryExpressionDSL(FromGatherer<R> fromGatherer, String tableAlias) {
+        this(fromGatherer);
+        tableAliases.put(table, tableAlias);
+    }
+    
+    public static <R> FromGatherer<R> select(SelectDSL<R> selectDSL, BasicColumn...selectList) {
+        return new FromGatherer.Builder<R>()
+                .withSelectList(selectList)
+                .withSelectDSL(selectDSL)
+                .build();
+    }
+    
+    public static <R> FromGatherer<R> selectDistinct(SelectDSL<R> selectDSL, BasicColumn...selectList) {
+        return new FromGatherer.Builder<R>()
+                .withSelectList(selectList)
+                .withSelectDSL(selectDSL)
+                .isDistinct()
+                .build();
+    }
+    
     public <T> QueryExpressionWhereBuilder where(BindableColumn<T> column, VisitableCondition<T> condition) {
         return new QueryExpressionWhereBuilder(column, condition);
     }
@@ -171,55 +190,58 @@ public SelectDSL<R>.FetchFirstFinisher fetchFirst(long fetchFirstRows) {
     }
 
     public static class FromGatherer<R> {
-        private FromGathererBuilder<R> builder;
-        private Map<SqlTable, String> tableAliasMap = new HashMap<>();
+        private String connector;
+        private BasicColumn[] selectList;
+        private SelectDSL<R> selectDSL;
+        private boolean isDistinct;
         private SqlTable table;
 
-        public FromGatherer(FromGathererBuilder<R> builder) {
-            this.builder = builder;
+        public FromGatherer(Builder<R> builder) {
+            this.connector = builder.connector;
+            this.selectList = Objects.requireNonNull(builder.selectList);
+            this.selectDSL = Objects.requireNonNull(builder.selectDSL);
+            this.isDistinct = builder.isDistinct;
         }
 
         public QueryExpressionDSL<R> from(SqlTable table) {
             this.table = table;
-            
             return new QueryExpressionDSL<>(this);
         }
 
         public QueryExpressionDSL<R> from(SqlTable table, String tableAlias) {
             this.table = table;
-            tableAliasMap.put(table, tableAlias);
-            return new QueryExpressionDSL<>(this);
-        }
-    }
-    
-    public static class FromGathererBuilder<R> {
-        private String connector;
-        private BasicColumn[] selectList;
-        private SelectDSL<R> selectDSL;
-        private boolean isDistinct;
-        
-        public FromGathererBuilder<R> withConnector(String connector) {
-            this.connector = connector;
-            return this;
-        }
-
-        public FromGathererBuilder<R> withSelectList(BasicColumn[] selectList) {
-            this.selectList = selectList;
-            return this;
-        }
-
-        public FromGathererBuilder<R> withSelectDSL(SelectDSL<R> selectDSL) {
-            this.selectDSL = selectDSL;
-            return this;
+            return new QueryExpressionDSL<>(this, tableAlias);
         }
 
-        public FromGathererBuilder<R> isDistinct() {
-            this.isDistinct = true;
-            return this;
-        }
-        
-        public FromGatherer<R> build() {
-            return new FromGatherer<>(this);
+        public static class Builder<R> {
+            private String connector;
+            private BasicColumn[] selectList;
+            private SelectDSL<R> selectDSL;
+            private boolean isDistinct;
+            
+            public Builder<R> withConnector(String connector) {
+                this.connector = connector;
+                return this;
+            }
+
+            public Builder<R> withSelectList(BasicColumn[] selectList) {
+                this.selectList = selectList;
+                return this;
+            }
+
+            public Builder<R> withSelectDSL(SelectDSL<R> selectDSL) {
+                this.selectDSL = selectDSL;
+                return this;
+            }
+            
+            public Builder<R> isDistinct() {
+                this.isDistinct = true;
+                return this;
+            }
+            
+            public FromGatherer<R> build() {
+                return new FromGatherer<>(this);
+            }
         }
     }
 
@@ -509,19 +531,19 @@ public UnionBuilder(String connector) {
         }
 
         public FromGatherer<R> select(BasicColumn...selectList) {
-            return new FromGathererBuilder<R>()
+            return new FromGatherer.Builder<R>()
                     .withConnector(connector)
                     .withSelectList(selectList)
                     .withSelectDSL(selectDSL)
                     .build();
         }
 
         public FromGatherer<R> selectDistinct(BasicColumn...selectList) {
-            return new FromGathererBuilder<R>()
+            return new FromGatherer.Builder<R>()
                     .withConnector(connector)
-                    .isDistinct()
                     .withSelectList(selectList)
                     .withSelectDSL(selectDSL)
+                    .isDistinct()
                     .build();
         }
     }

src/main/java/org/mybatis/dynamic/sql/select/SelectDSL.java

@@ -45,29 +45,14 @@ private SelectDSL(Function<SelectModel, R> adapterFunction) {
         buildDelegateMethod = this::internalBuild;
     }
 
-    private QueryExpressionDSL.FromGatherer<R> queryExpressionBuilder(BasicColumn...selectList) {
-        return new QueryExpressionDSL.FromGathererBuilder<R>()
-                .withSelectDSL(this)
-                .withSelectList(selectList)
-                .build();
-    }
-    
-    private QueryExpressionDSL.FromGatherer<R> distinctQueryExpressionBuilder(BasicColumn...selectList) {
-        return new QueryExpressionDSL.FromGathererBuilder<R>()
-                .withSelectDSL(this)
-                .withSelectList(selectList)
-                .isDistinct()
-                .build();
-    }
-    
     public static QueryExpressionDSL.FromGatherer<SelectModel> select(BasicColumn...selectList) {
         return select(Function.identity(), selectList);
     }
 
     public static <R> QueryExpressionDSL.FromGatherer<R> select(Function<SelectModel, R> adapterFunction,
             BasicColumn...selectList) {
-        SelectDSL<R> selectModelBuilder = new SelectDSL<>(adapterFunction);
-        return selectModelBuilder.queryExpressionBuilder(selectList);
+        SelectDSL<R> selectDSL = new SelectDSL<>(adapterFunction);
+        return QueryExpressionDSL.select(selectDSL, selectList);
     }
 
     public static QueryExpressionDSL.FromGatherer<SelectModel> selectDistinct(BasicColumn...selectList) {
@@ -76,8 +61,8 @@ public static QueryExpressionDSL.FromGatherer<SelectModel> selectDistinct(BasicC
 
     public static <R> QueryExpressionDSL.FromGatherer<R> selectDistinct(Function<SelectModel, R> adapterFunction,
             BasicColumn...selectList) {
-        SelectDSL<R> selectModelBuilder = new SelectDSL<>(adapterFunction);
-        return selectModelBuilder.distinctQueryExpressionBuilder(selectList);
+        SelectDSL<R> selectDSL = new SelectDSL<>(adapterFunction);
+        return QueryExpressionDSL.selectDistinct(selectDSL, selectList);
     }
 
     public static <T> QueryExpressionDSL.FromGatherer<MyBatis3SelectModelAdapter<T>> selectWithMapper(