Merge pull request #147 from oracle/fix/materialized_view

Fix ORA-12098: cannot comment on the materialized view

Author: aosingh

dbt/include/oracle/macros/adapters.sql

@@ -260,8 +260,12 @@
 
 {% macro oracle__alter_relation_comment(relation, comment) %}
   {% set escaped_comment = oracle_escape_comment(comment) %}
+  {% if relation.type == 'materialized_view' %}
+  comment on materialized view {{ relation }} is {{ escaped_comment }}
+  {% else %}
   {# "comment on table" even for views #}
   comment on table {{ relation }} is {{ escaped_comment }}
+  {% endif %}
 {% endmacro %}
 
 {% macro oracle__persist_docs(relation, model, for_relation, for_columns) -%}

dbt_adbs_test_project/models/sales_internet_mv.sql

@@ -13,6 +13,6 @@
   See the License for the specific language governing permissions and
   limitations under the License.
 #}
-{{ config(materialized='materialized_view')}}
+{{ config(materialized='materialized_view',  persist_docs={"relation": true, "columns": true}) }}
 select * from {{ source('sh_database', 'sales') }}
 where channel_id = 5

dbt_adbs_test_project/models/schema.yml

@@ -22,6 +22,9 @@ sources:
       - name: costs
 
 models:
+  - name: sales_internet_mv
+    description: Test comment for Materialized View
+
   - name: kafka
     description: kafka_description
     config: