Merge branch 'main' of github.com:dbt-labs/dbt-external-tables into p…

…r/thomas-vl/230

integration_tests/models/plugins/bigquery/bigquery_external.yml

@@ -67,6 +67,16 @@ sources:
         columns: *cols-of-the-people
         tests: *equal-to-the-people
 
+      - name: people_csv_with_max_staleness
+        external:
+          location: 'gs://dbt-external-tables-testing/csv/*'
+          options:
+            format: csv
+            skip_leading_rows: 1
+            max_staleness: INTERVAL 1 HOUR
+        columns: *cols-of-the-people
+        tests: *equal-to-the-people
+
       - name: people_csv_schema_auto_detect_expiration
         external:
           hours_to_expiration: 24

macros/plugins/bigquery/create_external_table.sql

@@ -5,6 +5,8 @@
     {%- set partitions = external.partitions -%}
     {%- set options = external.options -%}
     {%- set hours_to_expiration = external.get('hours_to_expiration') -%}
+    {%- set non_string_options = ['max_staleness', 'hours_to_expiration'] %}
+
     {% if options is mapping and options.get('connection_name', none) %}
         {% set connection_name = options.pop('connection_name') %}
     {% endif %}
@@ -38,7 +40,7 @@
             uris = [{%- for uri in uris -%} '{{uri}}' {{- "," if not loop.last}} {%- endfor -%}]
             {%- if options is mapping -%}
             {%- for key, value in options.items() if key != 'uris' %}
-                {%- if value is string -%}
+                {%- if value is string and key not in non_string_options -%}
                 , {{key}} = '{{value}}'
                 {%- else -%}
                 , {{key}} = {{value}}