Skip to content

Commit 1fa3ef0

Browse files
dtunikovclaude
andcommitted
drop self-named column aliases in e2e datetime range query
Aliasing toString(col) back to the column's own name makes older ClickHouse (LTS) resolve the expression against the alias and return empty; the test only reads columns positionally, so the aliases are unnecessary. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 24b996c commit 1fa3ef0

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

flow/e2e/clickhouse_mysql_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1809,7 +1809,7 @@ func (s ClickHouseSuite) Test_MySQL_DateTime_ClickHouse_Range() {
18091809
"d_null_low", "d_null_high", "d_nn_low", "d_nn_high", "d_ok",
18101810
"dt_null_low", "dt_null_high", "dt_nn_low", "dt_nn_high", "dt_ok",
18111811
} {
1812-
colExprs = append(colExprs, fmt.Sprintf("ifNull(toString(%s),'') AS %s", c, c))
1812+
colExprs = append(colExprs, fmt.Sprintf("ifNull(toString(%s),'')", c))
18131813
}
18141814
cols := strings.Join(colExprs, ",")
18151815
rows, err := s.GetRows(dstTableName, cols)

0 commit comments

Comments
 (0)