Merge pull request #34 from starburstdata/hovaesco/release-1.0.1

Release 1.0.1

.bumpversion-dbt.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 1.0.0
+current_version = 1.0.1
 parse = (?P<major>\d+)
 	\.(?P<minor>\d+)
 	\.(?P<patch>\d+)

.bumpversion.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 1.0.0
+current_version = 1.0.1
 parse = (?P<major>\d+)
 	\.(?P<minor>\d+)
 	\.(?P<patch>\d+)

Makefile

@@ -3,13 +3,13 @@ dbt-trino-tests:
 	./docker/dbt/build.sh
 	./docker/init_trino.bash
 	pip install -r dev_requirements.txt
-	tox || ./docker/remove_trino.bash
+	tox -r  || ./docker/remove_trino.bash
 	./docker/remove_trino.bash
 
 dbt-starburst-tests:
 	docker network create dbt-net || true
 	./docker/dbt/build.sh
 	./docker/init_starburst.bash
 	pip install -r dev_requirements.txt
-	tox || ./docker/remove_starburst.bash
+	tox -r || ./docker/remove_starburst.bash
 	./docker/remove_starburst.bash

README.md

@@ -25,7 +25,7 @@ This repository represents a fork of the [dbt-presto](https://github.com/dbt-lab
 
 ### Compatibility
 
-This dbt plugin has been tested against `Trino` version `366` and `Starburst Enterprise` version `365-e`.
+This dbt plugin has been tested against `Trino` version `368` and `Starburst Enterprise` version `368-e`.
 
 ## Installation
 
@@ -265,7 +265,7 @@ pip install -r dev_requirements.txt
 Run from the base directory of the project the command:
 
 ```sh
-tox
+tox -r
 ```
 
 or
@@ -284,8 +284,6 @@ bumpversion --config-file .bumpversion-dbt.cfg patch --new-version <new-version>
 bumpversion --config-file .bumpversion.cfg patch --new-version <new-version> --allow-dirty
 ```
 
-Additionally bump dbt-core version in dbt testing image `docker/dbt/Dockerfile`.
-
 Next step is to merge bump commit and making sure that test suite pass.
 
 Finally to release `dbt-trino` to PyPi and GitHub trigger release workflow `release.yml`.

dbt/adapters/trino/__version__.py

@@ -1 +1 @@
-version = "1.0.0"
+version = "1.0.1"

docker-compose-starburst.yml

@@ -3,7 +3,7 @@ services:
   trino:
     ports:
       - "8080:8080"
-    image: "starburstdata/starburst-enterprise:365-e"
+    image: "starburstdata/starburst-enterprise:368-e"
     volumes:
       - ./docker/starburst/etc:/etc/starburst
       - ./docker/starburst/catalog:/etc/starburst/catalog

docker-compose-trino.yml

@@ -3,7 +3,7 @@ services:
   trino:
     ports:
       - "8080:8080"
-    image: "trinodb/trino:366"
+    image: "trinodb/trino:368"
     volumes:
       - ./docker/trino/etc:/usr/lib/trino/etc:ro
       - ./docker/trino/catalog:/etc/trino/catalog

docker/dbt/Dockerfile

@@ -1,4 +1,4 @@
-FROM fishtownanalytics/dbt:1.0.0
+FROM python:3.9.9-slim-bullseye
 USER root
 
 ENV DBT_DIR /opt/dbt-trino

requirements.txt

@@ -1,2 +1,2 @@
-dbt-core==1.0.0
+dbt-core==1.0.1
 trino==0.309.0

setup.py

@@ -27,7 +27,7 @@ def _dbt_trino_version():
 package_version = _dbt_trino_version()
 description = """The trino adapter plugin for dbt (data build tool)"""
 
-dbt_version = "1.0.0"
+dbt_version = "1.0.1"
 # the package version should be the dbt version, with maybe some things on the
 # ends of it. (0.19.1 vs 0.19.1a1, 0.19.1.1, ...)
 if not package_version.startswith(dbt_version):

test/unit/test_adapter.py

@@ -95,7 +95,7 @@ def _get_tester_for(self, column_type):
             return agate.TimeDelta()
 
         for instance in agate_helper.DEFAULT_TYPE_TESTER._possible_types:
-            if type(instance) is column_type:
+            if isinstance(instance, column_type):
                 return instance
 
         raise ValueError(f"no tester for {column_type}")