Warn in rolling window pages that Redshift does not support month or year intervals #8252
Conversation
Please note that Redshift does not support intervals of type month or year.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 8 Ignored Deployments
|
github-actions
bot
added
the
pr:community
Please note that Redshift does not support intervals of type month or year.
| <WarningBox> | ||
|
|
||
| Please note that Redshift does not support intervals of type month or year. | ||
|
|
||
| </WarningBox> | ||
|
|
There was a problem hiding this comment.
@morgan-at-cube I don't think we need to put this warning on this particular page. If we do, then we'd probably need to have it at every page mentioning rolling_window which is probably not the way to go.
| <WarningBox> | ||
|
|
||
| Please note that Redshift does not support intervals of type month or year. | ||
|
|
||
| </WarningBox> | ||
|
|
There was a problem hiding this comment.
@morgan-at-cube Do we know how it breaks? E.g., what is the exact error message?
I think we can probably do the following:
- Add the error message and the note on the Redshift data source page.
- Have a more generic note on this page, e.g., "There are known limitations for
rolling_windowsupport when using Redhsift (link to the data source page)". It's gonna be more extendable this way.
There was a problem hiding this comment.
Hey @igorlukanin , getting back to you about this has been in my to-do list for ages, sorry about that! I don't have the error message unfortunately :( Do you think we can still add a generic error message to this page anyway?
|
I think this is obsolete because of #9489 |
Cool, thanks! |
3 participants
Add warning box to two pages: Please note that Redshift does not support intervals of type month or year.
Context: https://cubedevinc.slack.com/archives/C021C7NT7RT/p1715293531642229
Check List
Issue Reference this PR resolves
[For example #12]
Description of Changes Made (if issue reference is not provided)
[Description goes here]