You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2021/06/15 01:23:35 UTC

[GitHub] [beam] apilloud commented on pull request #15010: DO NOT MERGE: attempt to add Postgresql function library to Beam SQL

apilloud commented on pull request #15010:
URL: https://github.com/apache/beam/pull/15010#issuecomment-861100661


   CONVERT_TIMEZONE didn't get added until here:
   https://github.com/apache/calcite/commit/e8b08c490ab4270945c35f31846cd36b5788cc23#diff-e376f6424208a605e128ae7589ac2582e197699d6e15809f30d32926f0c49349R57
   
   I think you need Calcite 1.21?
   
   If this did work we might just turn it on. We do have a problem with a large surface area of built in functions, which is going to make pushing SQL down to runners more difficult, but adding more built-in functions is probably better than adding more config flags.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org