Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redshift shouldn't support window clause #1035

Closed
owenjonesuob opened this issue Oct 28, 2022 · 0 comments · Fixed by #1034
Closed

Redshift shouldn't support window clause #1035

owenjonesuob opened this issue Oct 28, 2022 · 0 comments · Fixed by #1034

Comments

@owenjonesuob
Copy link
Contributor

Currently, supports_window_clause() returns TRUE for Redshift connections, because the Redshift backend inherits from the Postgres backend, and Postgres does support window clauses.

However, Redshift does not currently support window clauses - at least, they are not mentioned anywhere in the AWS Redshift docs, and attempting to use a named window in a Redshift query in the same way as in a Postgres query, results in a syntax error.

Fixable by adding supports_window_clause() S3 methods for Redshift connections in backend-redshift.R.

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 a pull request may close this issue.

1 participant