Skip to content

Commit

Permalink
[fix](planner) fix current_timestamp param type mismatch when doing s…
Browse files Browse the repository at this point in the history
…tream load (#23092)

FileLoadScanNode did not analyze the default value expr, result in target param type int32 become int8 as the original IntLiteral type.
  • Loading branch information
TangSiyang2001 authored Aug 18, 2023
1 parent a8d63ef commit a7771ea
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -175,6 +175,7 @@ protected void setDefaultValueExprs(TableIf tbl,
if (column.getDefaultValue() != null) {
if (column.getDefaultValueExprDef() != null) {
expr = column.getDefaultValueExpr();
expr.analyze(analyzer);
} else {
expr = new StringLiteral(column.getDefaultValue());
}
Expand Down
16 changes: 8 additions & 8 deletions regression-test/data/correctness_p0/test_current_timestamp.out
Original file line number Diff line number Diff line change
Expand Up @@ -12,28 +12,28 @@
4

-- !stream_load_csv1 --
4
5

-- !stream_load_csv2 --
4
5

-- !stream_load_csv3 --
4
5

-- !stream_load_csv4 --
4
5

-- !stream_load_json1 --
2
3

-- !stream_load_json2 --
2
3

-- !stream_load_json3 --
2
3

-- !stream_load_json4 --
2
3

-- !stream_load_json5 --
9 apache
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
5,ee
6,ff
7,gg
8,hh
8,hh
9,

0 comments on commit a7771ea

Please sign in to comment.