Merge pull request #416 from ewanwalk/fix(SimpleAggregateFunctions)

Fix not supporting SimpleAggregateFunction types not working on types with more than one arguement
ClickHouse · Sep 17, 2021 · c47e24b · c47e24b
2 parents 1ae00d3 + aa7a49b
commit c47e24b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/lib/column/column.go b/lib/column/column.go
@@ -195,6 +195,11 @@ func getNestedType(chType string, wrapType string) (string, error) {
 		if len(nested) == 2 {
 			return strings.TrimSpace(nested[1]), nil
 		}
+
+		if len(nested) == 3 {
+			return strings.TrimSpace(strings.Join(nested[1:], ",")), nil
+		}
 	}
+
 	return "", fmt.Errorf("column: invalid %s type (%s)", wrapType, chType)
 }
diff --git a/lib/column/column_test.go b/lib/column/column_test.go
@@ -632,6 +632,7 @@ func Test_Column_SimpleAggregateFunc(t *testing.T) {
 		"SimpleAggregateFunction(anyLast, UInt8)":          "UInt8",
 		"SimpleAggregateFunction(anyLast, Nullable(IPv4))": "Nullable(IPv4)",
 		"SimpleAggregateFunction(max, Nullable(DateTime))": "Nullable(DateTime)",
+		"SimpleAggregateFunction(sum, Decimal(38, 8))":     "Decimal(38, 8)",
 	}
 
 	for key, val := range data {