Skip to content

Commit

Permalink
fix: Use derived ArrayDims instead of deprecated attndims (#3032)
Browse files Browse the repository at this point in the history
  • Loading branch information
hendrikhofstadt committed Dec 4, 2023
1 parent ff85353 commit 564f67d
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions internal/engine/postgresql/analyzer/analyze.go
Original file line number Diff line number Diff line change
Expand Up @@ -253,8 +253,7 @@ func (a *Analyzer) Analyze(ctx context.Context, n ast.Node, query string, migrat
if err != nil {
return nil, err
}
// TODO: Why are these dims different?
dt, isArray, _ := parseType(col.DataType)
dt, isArray, dims := parseType(col.DataType)
notNull := col.NotNull
name := field.Name
result.Columns = append(result.Columns, &core.Column{
Expand All @@ -263,7 +262,7 @@ func (a *Analyzer) Analyze(ctx context.Context, n ast.Node, query string, migrat
DataType: dt,
NotNull: notNull,
IsArray: isArray,
ArrayDims: int32(col.ArrayDims),
ArrayDims: int32(dims),
Table: &core.Identifier{
Schema: tbl.SchemaName,
Name: tbl.TableName,
Expand Down

0 comments on commit 564f67d

Please sign in to comment.