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

Remove the Redshift-specific logic for enable_case_sensitive_identifier #206

Closed
dbeatty10 opened this issue Nov 21, 2024 · 0 comments · Fixed by #208
Closed

Remove the Redshift-specific logic for enable_case_sensitive_identifier #206

dbeatty10 opened this issue Nov 21, 2024 · 0 comments · Fixed by #208
Labels
enhancement New feature or request

Comments

@dbeatty10
Copy link
Contributor

Describe the feature

#63 needed to introduce logic related to enable_case_sensitive_identifier because the cluster use for CI testing had the default setting of false. But now it has the setting of true, so we might be able to remove the Redshift-specific portions of that PR.

We should make sure to document that the CI tests for this package require a cluster with enable_case_sensitive_identifier=true.

Describe alternatives you've considered

N/A

Additional context

N/A

Who will this benefit?

What kind of use case will this feature be useful for? Please be specific and provide examples, this will help us prioritize properly.

Are you interested in contributing this feature?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant