duckdb / dbt-duckdb

dbt (http://getdbt.com) adapter for DuckDB (http://duckdb.org)
Apache License 2.0
882 stars 78 forks source link

Add a more helpful error message in the case that the options argument isn't a dictionary #250

Closed jwills closed 1 year ago

jwills commented 1 year ago

A couple of different folks have hit hard-to-diagnose errors when they pass in a string for the options field in external materializations; since I need to be able to use the partition_by field in order to correctly create a view for the materialized file(s), I need to keep the options field as a dictionary and provide a more helpful error message in the situation that another variable type (e.g., a string) is passed instead.