Conversation
Rename the ToDatetime so that it's inline with the other date(time) processing functions (DateTrunc, DateExtract etc.) `to_date` is thus now introduced and `to_datetime` is kept as an alias.
|
Documentation preview: |
|
Pinging @elastic/es-ql (Team:QL) |
|
Pinging @elastic/elasticsearch-esql (:Query Languages/ES|QL) |
|
I think we'll have to separate DATE_TIME and DATE even though we'll use the same representation for them. That said, I'm fine with this PR however it's worth raising this up with the group along side a list of functions that work only on DATE vs DATE_TIME (as we do in SQL). |
|
Taking a look at our SQL time functions, I think we can group them in some five categories:
The functions that take a Having (Fwiw, other significant pipe languages don't expose similar types.) In any case, with existing date_extract()/_format()/_parse()/_trunc(), we would need to align one one side or the other, IMO. |
|
Sync'd about it and decided to keep things as are: we might add a |
Rename the
ToDatetimetoToDateso that it is inline with the other date(time) processing functions (DateTrunc,DateExtractetc.)to_dateis thus now introduced andto_datetimeis kept as an alias.