search

alittlesliceoftom / insert_by_timeperiod

Dbt package for advanced materialisation "insert by timeperiod" which builds on insert by period implementations. Currently tested in Azure Synaspse only.
Apache License 2.0
2 stars 1 forks source link

Remove inside transaction statement to fix hooks #6

Closed alittlesliceoftom closed 1 month ago

alittlesliceoftom commented 1 month ago

Fixes #5

alittlesliceoftom commented 1 month ago

Note root code is here: https://github.com/dbt-labs/dbt-adapters/blob/8ff69435a753f0a94dd1274d0e8dad199805cd13/dbt/include/global_project/macros/materializations/hooks.sql#L4

For run_hooks, and the inside_transaction = False causes: commit;

To be put before the hook statement.

This seems to prevent execution in our case.