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

[staging] Throw error if model contract enforced #328

Merged
merged 1 commit into from
May 2, 2023

Conversation

susodapop
Copy link
Contributor

Resolves #

Description

Checklist

  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have updated the CHANGELOG.md and added information about my change to the "dbt-databricks next" section.

* Mirror of dbt-databricks#319 but rebased on 1.5.test branch

Signed-off-by: Jesse Whitehouse <jesse.whitehouse@databricks.com>

* Throw error if model contract enforced

Until we support dbt 1.5 model contracts we will throw an error if contract->enforced is set to true for a model.

Added doc outlining differences between dbt-databricks constraints and dbt model contract.

Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>

* Update change log

Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>

* added missing newline

Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>

* Add all columns to model definition

Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>

---------

Signed-off-by: Jesse Whitehouse <jesse.whitehouse@databricks.com>
Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>
Co-authored-by: Jesse Whitehouse <jesse.whitehouse@databricks.com>
@susodapop susodapop changed the base branch from main to 1.5.test May 1, 2023 20:19
@susodapop susodapop temporarily deployed to azure-prod-peco May 1, 2023 22:49 — with GitHub Actions Inactive
@susodapop susodapop temporarily deployed to azure-prod-pecou May 1, 2023 22:49 — with GitHub Actions Inactive
@rcypher-databricks rcypher-databricks merged commit 1d441d0 into 1.5.test May 2, 2023
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.

None yet

3 participants