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

full refresh behaviour fix #107

Merged
merged 1 commit into from
Oct 27, 2023
Merged

full refresh behaviour fix #107

merged 1 commit into from
Oct 27, 2023

Conversation

ilikutle
Copy link
Collaborator

Fixes the problem introduced in #105

  • uses macro should_full_refresh when checking for the full refresh condition in incremental model creation

Copy link
Owner

@tglunde tglunde left a comment

Choose a reason for hiding this comment

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

tested ok.

@tglunde tglunde merged commit 0ad5784 into master Oct 27, 2023
tglunde pushed a commit that referenced this pull request Oct 27, 2023
Co-authored-by: ilikutle <ilija.kutle@alligator-company.com>
tglunde added a commit that referenced this pull request Oct 27, 2023
peterkioko pushed a commit to GrantStreetGroup/dbt-exasol that referenced this pull request Nov 10, 2023
Co-authored-by: ilikutle <ilija.kutle@alligator-company.com>
peterkioko pushed a commit to GrantStreetGroup/dbt-exasol that referenced this pull request Nov 10, 2023
Merge in GITHUB/dbt-exasol from pull_in_upstream_changes to master

* commit '2f99f8e37333cb91df0f4700269e34fcfdb1febd':
  xdist for parallel pytest execution added - mv & clone miss in readme (tglunde#108)
  95 dbt16 tests  (tglunde#99)
  Bump urllib3 from 1.26.16 to 1.26.18 (tglunde#106)
  Bump cryptography from 41.0.3 to 41.0.4 (tglunde#100)
  caching fix, removed table macro overwrite (tglunde#103)
  full refresh behaviour fix (tglunde#107)
  adding debug test (tglunde#98)
  version bump 16 && limit simple test added (tglunde#96)
  bugfix null handling csv hash test - version upgrade 1.6 (tglunde#97)
  poetry update && re-added 4ec4415 for adding multiple cols in exasol (tglunde#93)
  v1.5 tests, incremental merge, temp table fix (tglunde#86)
  89 contract tests fail after dbt core update from 151 to 152 (tglunde#92)
  source freshness timestamp parsing fix (tglunde#91)
  v1.5 model contracts (tglunde#87)
  Bump sqlfluff from 2.1.1 to 2.1.2 (tglunde#83)
  tglunde#80 Disable view comments outside of DDL (tglunde#82)
  adding parameter for OpenID login with access/refresh token (tglunde#79)
  Bump requests from 2.29.0 to 2.31.0 (tglunde#76)
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