Merge pull request #39 from oracle/feature/dbt-core1.2

Support for dbt-core 1.2

Author: aosingh

.github/workflows/oracle-xe-adapter-tests.yml

@@ -45,7 +45,7 @@ jobs:
       - name: Install dbt-oracle with core dependencies
         run: |
           python -m pip install --upgrade pip
-          pip install pytest dbt-tests-adapter==1.1.2
+          pip install pytest dbt-tests-adapter==1.2.0
           pip install -r requirements.txt
           pip install -e .
 

.gitignore

@@ -142,4 +142,4 @@ doc/build.gitbak
 .venv1.1.0
 .venv
 .bldenv
-
+.venv1.2/

Makefile

@@ -1,5 +1,5 @@
 # Configuration variables
-VERSION=1.1.2
+VERSION=1.2.0
 PROJ_DIR?=$(shell pwd)
 VENV_DIR?=${PROJ_DIR}/.bldenv
 BUILD_DIR=${PROJ_DIR}/build

dbt/adapters/oracle/__version__.py

@@ -14,4 +14,4 @@
   See the License for the specific language governing permissions and
   limitations under the License.
 """
-version = "1.1.2"
+version = "1.2.0"

dbt/include/oracle/macros/adapters.sql

@@ -58,7 +58,7 @@
                                 'SYNONYM',
                                 'MATERIALIZED VIEW'
                               )
-                      AND owner = '{{ schema | upper }}')
+                      AND upper(owner) = '{{ schema | upper }}')
     LOOP
         BEGIN
           IF cur_rec.object_type = 'TABLE'
@@ -130,8 +130,8 @@
   {% call statement('get_columns_in_relation', fetch_result=True) %}
       with columns as (
         select
-            UPPER(SYS_CONTEXT('userenv', 'DB_NAME')) table_catalog,
-            UPPER(owner) table_schema,
+            SYS_CONTEXT('userenv', 'DB_NAME') table_catalog,
+            owner table_schema,
             table_name,
             column_name,
             data_type,
@@ -178,9 +178,9 @@
           numeric_precision as "numeric_precision",
           numeric_scale as "numeric_scale"
       from columns
-      where table_name = upper('{{ relation.identifier }}')
+      where upper(table_name) = upper('{{ relation.identifier }}')
         {% if relation.schema %}
-        and table_schema = upper('{{ relation.schema }}')
+        and upper(table_schema) = upper('{{ relation.schema }}')
         {% endif %}
       order by ordinal_position
 
@@ -314,8 +314,8 @@
 {% macro oracle__list_relations_without_caching(schema_relation) %}
   {% call statement('list_relations_without_caching', fetch_result=True) -%}
     with tables as
-      (select UPPER(SYS_CONTEXT('userenv', 'DB_NAME')) table_catalog,
-         UPPER(owner) table_schema,
+      (select SYS_CONTEXT('userenv', 'DB_NAME') table_catalog,
+         owner table_schema,
          table_name,
          case
            when iot_type = 'Y'
@@ -326,8 +326,8 @@
          end table_type
        from sys.all_tables
        union all
-       select UPPER(SYS_CONTEXT('userenv', 'DB_NAME')),
-         UPPER(owner),
+       select SYS_CONTEXT('userenv', 'DB_NAME'),
+         owner,
          view_name,
          'VIEW'
        from sys.all_views
@@ -341,7 +341,7 @@
     end as "kind"
   from tables
   where table_type in ('BASE TABLE', 'VIEW')
-    and table_schema = upper('{{ schema_relation.schema }}')
+    and upper(table_schema) = upper('{{ schema_relation.schema }}')
   {% endcall %}
   {{ return(load_result('list_relations_without_caching').table) }}
 {% endmacro %}

dbt/include/oracle/macros/catalog.sql

@@ -71,8 +71,8 @@
               from sys.all_tab_columns
           ),
           tables as
-                (select UPPER(SYS_CONTEXT('userenv', 'DB_NAME')) table_catalog,
-                   UPPER(owner) table_schema,
+                (select SYS_CONTEXT('userenv', 'DB_NAME') table_catalog,
+                   owner table_schema,
                    table_name,
                    case
                      when iot_type = 'Y'
@@ -117,7 +117,7 @@
               and all_col_comments.column_name = columns.column_name
           where (
               {%- for schema in schemas -%}
-                tables.table_schema = upper('{{ schema }}'){%- if not loop.last %} or {% endif -%}
+                upper(tables.table_schema) = upper('{{ schema }}'){%- if not loop.last %} or {% endif -%}
               {%- endfor -%}
             )
           order by

requirements.txt

@@ -1,4 +1,4 @@
-dbt-core==1.1.2
+dbt-core==1.2.0
 cx_Oracle==8.3.0
 oracledb==1.0.3
 

requirements_dev.txt

@@ -6,4 +6,4 @@ tox
 coverage
 twine
 pytest
-dbt-tests-adapter==1.1.2
+dbt-tests-adapter==1.2.0

setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = dbt-oracle
-version = 1.1.2
+version = 1.2.0
 description = dbt (data build tool) adapter for the Oracle database
 long_description = file: README.md
 long_description_content_type = text/markdown
@@ -32,12 +32,12 @@ zip_safe = False
 packages = find:
 include_package_data = True
 install_requires =
-    dbt-core==1.1.2
+    dbt-core==1.2.0
     cx_Oracle==8.3.0
     oracledb==1.0.3
 test_suite=tests
 test_requires =
-    dbt-tests-adapter==1.1.2
+    dbt-tests-adapter==1.2.0
     pytest
 scripts =
     bin/create-pem-from-p12

setup.py

@@ -32,13 +32,13 @@
 
 
 requirements = [
-        "dbt-core==1.1.2",
+        "dbt-core==1.2.0",
         "cx_Oracle==8.3.0",
         "oracledb==1.0.3"
 ]
 
 test_requirements = [
-    "dbt-tests-adapter==1.1.2",
+    "dbt-tests-adapter==1.2.0",
     "pytest"
 ]
 
@@ -52,7 +52,7 @@
 
 url = 'https://github.com/oracle/dbt-oracle'
 
-VERSION = '1.1.2'
+VERSION = '1.2.0'
 setup(
     author="Oracle",
     python_requires='>=3.7.2',