adapt readme; add default materialization config

duckdb · Oct 25, 2023 · b48be37 · b48be37
1 parent 6ec7466
commit b48be37
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/dbt/adapters/duckdb/environments/local.py b/dbt/adapters/duckdb/environments/local.py
@@ -115,7 +115,7 @@ def load_source(self, plugin_name: str, source_config: utils.SourceConfig):
         df = plugin.load(source_config)
         assert df is not None
 
-        materialization = source_config.meta.get("materialization", "table")
+        materialization = source_config.meta.get("materialization", plugin.default_materialization())
         source_table_name = source_config.table_name()
         df_name = source_table_name.replace(".", "_") + "_df"
 

diff --git a/dbt/adapters/duckdb/plugins/__init__.py b/dbt/adapters/duckdb/plugins/__init__.py
@@ -121,3 +121,6 @@ def configure_cursor(self, cursor):
         :param cursor: A DuckDBPyConnection instance to be configured.
         """
         pass
+
+    def default_materialization(self):
+        return "table"
diff --git a/dbt/adapters/duckdb/plugins/delta.py b/dbt/adapters/duckdb/plugins/delta.py
@@ -40,6 +40,8 @@ def load(self, source_config: SourceConfig):
 
         return df
 
+    def default_materialization(self):
+        return "view"
 
 # Future
 # TODO add databricks catalog