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

Unable to override macros in adapter #2548

Closed
1 of 5 tasks
mikaelene opened this issue Jun 15, 2020 · 3 comments · Fixed by #2590
Closed
1 of 5 tasks

Unable to override macros in adapter #2548

mikaelene opened this issue Jun 15, 2020 · 3 comments · Fixed by #2590
Labels
bug Something isn't working

Comments

@mikaelene
Copy link
Contributor

mikaelene commented Jun 15, 2020

Describe the bug

Some of the standard dbt macros can't be overridden by an adapter. I.e. Test, snapshots. This is present since dbt v0.16.0

Steps To Reproduce

Create an adapter the aims to override the standard macro snapshot_check_strategy. This has made it impossible for the dbt-sqlserver adapter to follow along with newer dbt verison.

Expected behavior

Expects that the macro is overridden by the adapter version.

Screenshots and log output

If applicable, add screenshots or log output to help explain your problem.

System information

Which database are you using dbt with?

  • postgres
  • redshift
  • bigquery
  • snowflake
  • other (specify: SQL Server

The output of dbt --version:

installed version: 0.17.0

The operating system you're using:
Osx and windows

The output of python --version:
Python 3.7.7

Additional context

Add any other context about the problem here.

@mikaelene mikaelene added bug Something isn't working triage labels Jun 15, 2020
@drewbanin drewbanin removed the triage label Jun 16, 2020
@drewbanin drewbanin modified the milestones: Marian Anderson, 0.17.1 Jun 16, 2020
@drewbanin
Copy link
Contributor

thanks for opening this issue @mikaelene! The issue you ran into specifically was around overriding the snapshot_check_strategy macro specifically, right? Or were there other specific macros that you were unable to override in the dbt-sqlserver plugin?

@mikaelene
Copy link
Contributor Author

it was specifically snapshot_check_strategy. But I had the same issue for all tests macros in 0.16. I solved that with a workaround solution in #2414. But I think that there is probably a general problem that some macros can't be overridden since 0.16

@drewbanin drewbanin modified the milestones: 0.17.1, Marian Anderson Jun 18, 2020
beckjake added a commit that referenced this issue Jul 1, 2020
@dataders
Copy link
Contributor

@mikaelene looks like the bugfix is slated for 0.18.0 #2590
Perhaps I'll pull dbt's dev/master and see if things are working properly?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants