Add support for EXTRACT function in Postgres v14 Upgrade#1243
Add support for EXTRACT function in Postgres v14 Upgrade#1243yaojiejia wants to merge 4 commits into
Conversation
|
Sorry for the delay in reviewing. I have been traveling and did not see this PR. It seems given that |
Sorry, something went wrong.
Sounds good, just made some changes about that |
Sorry, something went wrong.
mrigger
left a comment
There was a problem hiding this comment.
LGTM, thanks!
Sorry, something went wrong.
|
I just noticed that there is a test failure (" time zone "g+zvi" not recognized"). Could you run the changed implementation for longer to check whether there are other errors that need to be added as expected errors? |
Sorry, something went wrong.
Closes #1242