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

fix 409 duplicate batch issue #1020

Merged
merged 19 commits into from
Nov 12, 2023
Merged

fix 409 duplicate batch issue #1020

merged 19 commits into from
Nov 12, 2023

Conversation

colin-rogers-dbt
Copy link
Contributor

replace deterministic default batch_id with uuid

resolves #1006
docs dbt-labs/docs.getdbt.com/#

Problem

Solution

Checklist

  • I have read the contributing guide and understand what's expected of me
  • 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
  • This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

@colin-rogers-dbt colin-rogers-dbt self-assigned this Nov 11, 2023
@colin-rogers-dbt colin-rogers-dbt requested a review from a team as a code owner November 11, 2023 23:11
@cla-bot cla-bot bot added the cla:yes label Nov 11, 2023
@colin-rogers-dbt colin-rogers-dbt merged commit 0c5422c into main Nov 12, 2023
21 checks passed
@colin-rogers-dbt colin-rogers-dbt deleted the fix17PyModelRegression branch November 12, 2023 17:59
@josh-acuitymd
Copy link

Hello!! Any idea when this fix will get released? Or at least guidance on a version that doesn't have this issue we can downgrade to?

github-actions bot pushed a commit that referenced this pull request Dec 12, 2023
* use dynamic schema in test_grant_access_to.py

* use dynamic schema in test_grant_access_to.py

* revert setup

* fix partitioninb

* skip TestPythonBatchIdModels

* add changie

* run python integration tests separately

* run python integration tests separately

* cleanup _get_batch_id

* add space to pipe

* fix integration.yml

* add extra changie

* use dynamic schema in test_grant_access_to.py

* use dynamic schema in test_grant_access_to.py

* revert setup

* replace deterministic default batch_id with uuid

* add changie

(cherry picked from commit 0c5422c)
colin-rogers-dbt added a commit that referenced this pull request Dec 13, 2023
* use dynamic schema in test_grant_access_to.py

* use dynamic schema in test_grant_access_to.py

* revert setup

* fix partitioninb

* skip TestPythonBatchIdModels

* add changie

* run python integration tests separately

* run python integration tests separately

* cleanup _get_batch_id

* add space to pipe

* fix integration.yml

* add extra changie

* use dynamic schema in test_grant_access_to.py

* use dynamic schema in test_grant_access_to.py

* revert setup

* replace deterministic default batch_id with uuid

* add changie

(cherry picked from commit 0c5422c)

Co-authored-by: colin-rogers-dbt <[email protected]>
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.

[ADAP-1016] [Regression] [1.7] python models don't work
3 participants