Skip to content

Commit

Permalink
[fix](planner) ctas update datev1 to datev2 should use equals (#28641)
Browse files Browse the repository at this point in the history
  • Loading branch information
morrySnow authored Dec 20, 2023
1 parent 2378478 commit 0ae87e7
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -89,11 +89,11 @@ public void analyze(Analyzer analyzer) throws UserException {
queryStmt.getResultExprs().get(i).getSrcSlotRef().getDesc().setColumn(columnCopy);
}
if (Config.enable_date_conversion) {
if (queryStmt.getResultExprs().get(i).getType() == Type.DATE) {
if (queryStmt.getResultExprs().get(i).getType().isDate()) {
Expr castExpr = queryStmt.getResultExprs().get(i).castTo(Type.DATEV2);
queryStmt.getResultExprs().set(i, castExpr);
}
if (queryStmt.getResultExprs().get(i).getType() == Type.DATETIME) {
if (queryStmt.getResultExprs().get(i).getType().isDatetime()) {
Expr castExpr = queryStmt.getResultExprs().get(i).castTo(Type.DATETIMEV2);
queryStmt.getResultExprs().set(i, castExpr);
}
Expand Down

0 comments on commit 0ae87e7

Please sign in to comment.