Skip to content

Fix: use CONCAT with CAST to create interval for DATETIME_ADD - #1847

Open
flybitcone wants to merge 1 commit into
MIT-LCP:mainfrom
flybitcone:main
Open

Fix: use CONCAT with CAST to create interval for DATETIME_ADD#1847
flybitcone wants to merge 1 commit into
MIT-LCP:mainfrom
flybitcone:main

Conversation

@flybitcone

Copy link
Copy Markdown

Change the interval and CROSS JOIN function

@flybitcone

Copy link
Copy Markdown
Author

The check is not pass may be due to the Python version in the test environment.

Pls check the error in detain:
Run actions/setup-python@v2
Version 3.7 was not found in the local cache
Error: Version 3.7 with arch x64 not found

Thanks.

@Chessing234 Chessing234 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this file is marked auto-generated — usually you want to fix the bigquery source and re-transpile rather than hand-edit concepts_postgres. also the patch drops the mimiciv_derived. schema and brings in GENERATE_ARRAY / DATETIME_ADD, which won't run as plain postgres. if the goal is interval construction, maybe keep GENERATE_SERIES + + INTERVAL '1' HOUR like the current transpiled version?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants