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

Throw error if model contract enforced #322

Merged
merged 5 commits into from
May 1, 2023

Conversation

rcypher-databricks
Copy link
Collaborator

@rcypher-databricks rcypher-databricks commented Apr 27, 2023

Description

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.

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.

Signed-off-by: Jesse Whitehouse <jesse.whitehouse@databricks.com>
@susodapop susodapop changed the base branch from main to make-integration-tests-work April 28, 2023 17:24
@susodapop susodapop changed the base branch from make-integration-tests-work to main April 28, 2023 17:29
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>
Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>
Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>
@rcypher-databricks rcypher-databricks changed the base branch from main to make-integration-tests-work April 28, 2023 17:32
Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>
@susodapop susodapop changed the base branch from make-integration-tests-work to staging-322 May 1, 2023 20:18
Copy link

@susodapop susodapop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is ready to merge to the staging branch so we can run the integration tests.

@susodapop susodapop merged commit 476e510 into databricks:staging-322 May 1, 2023
andrefurlan-db pushed a commit that referenced this pull request May 2, 2023
* 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>
rcypher-databricks added a commit to rcypher-databricks/dbt-databricks that referenced this pull request May 8, 2023
* 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>
rcypher-databricks added a commit to rcypher-databricks/dbt-databricks that referenced this pull request May 8, 2023
* 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>
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.

2 participants