Merge pull request #30 from fishtown-analytics/bump/dbt-0-17-0

Bump/dbt 0 17 0

Author: sanjanasen96

.circleci/config.yml

@@ -4,7 +4,7 @@ version: 2.1
 jobs:
   build:
     docker:
-      - image: circleci/python:3.6.2-stretch
+      - image: circleci/python:3.6.3-stretch
       - image: circleci/postgres:9.6.5-alpine-ram
 
     steps:
@@ -20,7 +20,7 @@ jobs:
             . venv/bin/activate
 
             pip install --upgrade pip setuptools
-            pip install dbt==0.16.1
+            pip install dbt
 
             mkdir -p ~/.dbt
             cp integration_tests/ci/sample.profiles.yml ~/.dbt/profiles.yml

.github/ISSUE_TEMPLATE/CODEOWNERS

@@ -0,0 +1 @@
+*       @clrcrl

README.md

@@ -5,7 +5,7 @@
 > this package, especially as a post-hook. Please consider if this package is
 > appropriate for your use case before using it.
 
-Requires dbt >= 0.16.0
+Requires dbt >= 0.17.0
 
 This package provides out-of-the-box functionality to log events for all dbt
 invocations, including run start, run end, model start, and model end. It
@@ -63,3 +63,6 @@ New columns were added in v0.2.0:
 -   **event_is_full_refresh as is_full_refresh** - `boolean` whether the DBT run was a full refresh
 
 These will be added to your existing audit table automatically in the `on-run-start` DBT hook, and added to the staging tables deployed by this table when they are ran. The existing `event_schema` column will also be propagated into to `stg_dbt_model_deployments` as `schema`.
+
+###Contributing
+Additional contributions to this repo are very welcome! Check out [this](https://discourse.getdbt.com/t/contributing-to-an-external-dbt-package/657) post on the best workflow for contributing to a package. All PRs should only include functionality that is contained within all Segment deployments; no implementation-specific details should be included.

dbt_project.yml

@@ -1,5 +1,8 @@
 name: 'logging'
-version: '1.0'
+version: '0.3.0'
+config-version: 2
+
+require-dbt-version: [">=0.17.0"]
 
 source-paths: ["models"]
 analysis-paths: ["analysis"]
@@ -27,4 +30,4 @@ on-run-end:
 
 models:
   logging:
-    schema: meta
+    +schema: meta

integration_tests/dbt_project.yml

@@ -1,10 +1,10 @@
 
 name: 'event_logging_integration_tests'
-version: '1.0'
+version: '0.3.0'
+config-version: 2
 
 profile: 'integration_tests'
 
-require-dbt-version: ">=0.15.0"
 
 models:
   pre-hook: "{{ logging.log_model_start_event() }}"

integration_tests/macros/drop_audit_schema.sql

@@ -2,7 +2,8 @@
     {% set audit_schema=logging.get_audit_schema() %}
 
     {% if adapter.check_schema_exists(target.database, audit_schema) %}
-        {% do drop_schema(target.database, audit_schema) %}
+        {% set audit_schema_relation = api.Relation.create(database=target.database, schema=audit_schema).without_identifier() %}
+        {% do drop_schema(audit_schema_relation) %}
         {% do run_query("commit;") %}
         {{ dbt_utils.log_info("Audit schema dropped")}}
 

packages.yml

@@ -1,3 +1,3 @@
 packages:
  - package: fishtown-analytics/dbt_utils
-   version: [">=0.2.4", "<0.4.0"]
+   version: [">=0.2.4", "<0.5.0"]