jakevin

Results 58 comments of jakevin

@Dandandan . Hi, I'm a little confused about the details. Specifically how to unify `NamedExpr` and `Expr`. I think option is a good way. -------- Now alias in `Expr`: Expr...

😅I found that it isn't easy problem, I need to think more about it. `Expr` is used in many places.

`Aliases` in SQL is just used in `Table` and `Column` in the `fromClause` and `selectClause`. So, like `Aggregate`, `aggr_expr` is `NamedExpr`, but `group_expr` isn't.

In addition, I find some `Filter Expr` use `Alias` after rewrite which make `logicplan` can be connected after rewrited. It's related with #1319 #1316 It mayn't a good way by...

I prepare to handle it, If there is any progress, I'll record it under this issue

**Insert** `Insert timestamp` is supported at present, so we can ignore `insert`. **store** It isn't good idea that timestamp store as `String` in special format . **return/show** Naturally, we will...

I think it isn't wise that add a new type or modify many code due to this demand.