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

Modify adapter to support unified constraint fields #372

Merged
merged 9 commits into from
Mar 22, 2023

Conversation

peterallenwebb
Copy link
Contributor

resolves #341

Description

Modifies this adapter to support the now-unified constraint fields on columns

Checklist

@peterallenwebb peterallenwebb requested a review from a team as a code owner March 15, 2023 15:43
@cla-bot cla-bot bot added the cla:yes label Mar 15, 2023
@mikealfare mikealfare self-requested a review March 15, 2023 20:22
dbeatty10 and others added 4 commits March 17, 2023 16:31
* add triage-labels workflow

* fix end of files
…ersions. (#371)

Co-authored-by: Mila Page <versusfacit@users.noreply.github.com>
@peterallenwebb peterallenwebb merged commit f0d92d2 into main Mar 22, 2023
@peterallenwebb peterallenwebb deleted the paw/ct-2214-unify-constraints branch March 22, 2023 17:29
abbywh pushed a commit to abbywh/dbt-redshift that referenced this pull request Oct 11, 2023
* CT-2214: Modify adapter to work with unified constraint fields

* CT-2214: Add changelog entry

* CT-2214: Add temporary branch pin

* add triage-labels workflow (dbt-labs#366)

* add triage-labels workflow

* fix end of files

* Add tests to precommit hooks and so close out this repo for test conversions. (dbt-labs#371)

Co-authored-by: Mila Page <versusfacit@users.noreply.github.com>

* CT-2214: Remove unneeded line

* CT-2214: Revert requirements in preparation for merge

---------

Co-authored-by: Doug Beatty <44704949+dbeatty10@users.noreply.github.com>
Co-authored-by: Mila Page <67295367+VersusFacit@users.noreply.github.com>
Co-authored-by: Mila Page <versusfacit@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CT-2214] [Redshift] Use new ColumnLevelConstraint structure in redshift__get_columns_spec_ddl
5 participants