fix: order by fields must be in group by (compatible for Dameng)

Author: Qiu Jian

querydefs.go

@@ -195,11 +195,27 @@ func queryString(tq *SQuery, tmpFields ...IQueryField) string {
 	}
 	if tq.groupBy != nil && len(tq.groupBy) > 0 {
 		buf.WriteString(" GROUP BY ")
-		for i, f := range tq.groupBy {
+		groupByFields := make(map[string]IQueryField)
+		for i := range tq.groupBy {
+			f := tq.groupBy[i]
+			if _, ok := groupByFields[f.Name()]; ok {
+				continue
+			}
 			if i > 0 {
 				buf.WriteString(", ")
 			}
 			buf.WriteString(f.Reference())
+			groupByFields[f.Name()] = f
+		}
+		// DAMENG SQL Compatibility, all order by fields should be in group by
+		for i := range tq.orderBy {
+			f := tq.orderBy[i]
+			if _, ok := groupByFields[f.field.Name()]; ok {
+				continue
+			}
+			buf.WriteString(", ")
+			buf.WriteString(f.field.Reference())
+			groupByFields[f.field.Name()] = f.field
 		}
 	}
 	/*if tq.having != nil {
@@ -208,7 +224,8 @@ func queryString(tq *SQuery, tmpFields ...IQueryField) string {
 	}*/
 	if tq.orderBy != nil && len(tq.orderBy) > 0 {
 		buf.WriteString(" ORDER BY ")
-		for i, f := range tq.orderBy {
+		for i := range tq.orderBy {
+			f := tq.orderBy[i]
 			if i > 0 {
 				buf.WriteString(", ")
 			}