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

v1.5 tests, incremental merge, temp table fix #86

Merged
merged 22 commits into from
Sep 1, 2023
Merged

Conversation

ilikutle
Copy link
Collaborator

#74 Added tests to support v1.5.
Added incremental merge strategy
#85 Fixed not deleting temp_table behaviour when using incremental model

@ilikutle ilikutle requested a review from tglunde August 10, 2023 13:02
@ilikutle ilikutle self-assigned this Aug 10, 2023
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.

code changes look ok, except one question regarding dictionary access all vs. user view/table in easol.

dbt/include/exasol/macros/adapters.sql Outdated Show resolved Hide resolved
@tglunde tglunde merged commit 3260909 into master Sep 1, 2023
@tglunde tglunde deleted the feature/tests-v1.5 branch November 3, 2023 01:14
peterkioko pushed a commit to GrantStreetGroup/dbt-exasol that referenced this pull request Nov 10, 2023
* simple seed tests

* namings fix

* aliases test

* grants test moved under adapter

* testStoreTestFailures added

* simple copy tests added

* simple copy tests fix

* simple copy test fix

* restructure and new tests

* persist docs and snapshot tests

* incremental merge, tests and version change

* redundant truncate removed

* merge conflict for adapters.sql resolved

* switched to exa_all metadata tables

* pyyaml fix 6.0.1

* added additional parameter introduced by dbt-labs after 1.5.0

* parameter name according to dbt-core

---------

Co-authored-by: ilikutle <ilija.kutle@alligator-company.com>
Co-authored-by: Torsten Glunde <torsten.glunde@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