Jiangling Tang

> Actually it would be great to add support of the DateTime() type, @amoskaliov #10537 will be tracked to support `DateTime*(timezone)`, you can keep an eye on it or I...

https://github.com/trinodb/trino/runs/8058855336?check_suite_focus=true

> I assume the motivation is to cleanup some code since we move from one EOL version to another EOL version? IIUC the main purpose is to support escaping comment...

(Change 2nd commit title)

(squashed commit)

Thanks for the review, I'll address these comments this weekend.

> It'd be useful to take inspiration from probably PostgreSQL type mapping tests for some useful values to test. I'll take a look and make them both as consistent as...

Close this issue and thank you all for making this happen. Since Redshift still has no type mapping tests, no type mapping documentation will be added for now (see https://github.com/trinodb/trino/pull/13832#issuecomment-1282942980).

Thanks for the reminder, we already knew that (see https://github.com/trinodb/trino/pull/11148#issuecomment-1216558987) and I will continue this work.

https://github.com/trinodb/trino/runs/7946669571?check_suite_focus=true