fix!: materialized_view require_partition_filter moved to top

Author: bdashrad

README.md

@@ -197,7 +197,7 @@ This module provisions a dataset and a list of tables with associated JSON schem
 | encryption\_key | Default encryption key to apply to the dataset. Defaults to null (Google-managed). | `string` | `null` | no |
 | external\_tables | A list of objects which include table\_id, expiration\_time, external\_data\_configuration, and labels. | <pre>list(object({<br>    table_id              = string,<br>    description           = optional(string),<br>    autodetect            = bool,<br>    compression           = string,<br>    ignore_unknown_values = bool,<br>    max_bad_records       = number,<br>    schema                = string,<br>    source_format         = string,<br>    source_uris           = list(string),<br>    csv_options = object({<br>      quote                 = string,<br>      allow_jagged_rows     = bool,<br>      allow_quoted_newlines = bool,<br>      encoding              = string,<br>      field_delimiter       = string,<br>      skip_leading_rows     = number,<br>    }),<br>    google_sheets_options = object({<br>      range             = string,<br>      skip_leading_rows = number,<br>    }),<br>    hive_partitioning_options = object({<br>      mode              = string,<br>      source_uri_prefix = string,<br>    }),<br>    expiration_time     = string,<br>    max_staleness       = optional(string),<br>    deletion_protection = optional(bool),<br>    labels              = map(string),<br>  }))</pre> | `[]` | no |
 | location | The regional location for the dataset only US and EU are allowed in module | `string` | `"US"` | no |
-| materialized\_views | A list of objects which includes view\_id, view\_query, clustering, time\_partitioning, range\_partitioning, expiration\_time and labels | <pre>list(object({<br>    view_id             = string,<br>    description         = optional(string),<br>    query               = string,<br>    enable_refresh      = bool,<br>    refresh_interval_ms = string,<br>    clustering          = list(string),<br>    time_partitioning = object({<br>      expiration_ms            = string,<br>      field                    = string,<br>      type                     = string,<br>      require_partition_filter = bool,<br>    }),<br>    range_partitioning = object({<br>      field = string,<br>      range = object({<br>        start    = string,<br>        end      = string,<br>        interval = string,<br>      }),<br>    }),<br>    expiration_time = string,<br>    max_staleness   = optional(string),<br>    labels          = map(string),<br>  }))</pre> | `[]` | no |
+| materialized\_views | A list of objects which includes view\_id, view\_query, clustering, time\_partitioning, range\_partitioning, expiration\_time and labels | <pre>list(object({<br>    view_id             = string,<br>    description         = optional(string),<br>    query               = string,<br>    enable_refresh      = bool,<br>    refresh_interval_ms = string,<br>    clustering          = list(string),<br>    require_partition_filter = optional(bool),<br>    time_partitioning = object({<br>      expiration_ms            = string,<br>      field                    = string,<br>      type                     = string,<br>    }),<br>    range_partitioning = object({<br>      field = string,<br>      range = object({<br>        start    = string,<br>        end      = string,<br>        interval = string,<br>      }),<br>    }),<br>    expiration_time = string,<br>    max_staleness   = optional(string),<br>    labels          = map(string),<br>  }))</pre> | `[]` | no |
 | max\_time\_travel\_hours | Defines the time travel window in hours | `number` | `null` | no |
 | project\_id | Project where the dataset and table are created | `string` | n/a | yes |
 | resource\_tags | A map of resource tags to add to the dataset | `map(string)` | `{}` | no |

main.tf

@@ -130,25 +130,25 @@ resource "google_bigquery_table" "view" {
 }
 
 resource "google_bigquery_table" "materialized_view" {
-  for_each            = local.materialized_views
-  dataset_id          = google_bigquery_dataset.main.dataset_id
-  friendly_name       = each.key
-  table_id            = each.key
-  description         = each.value["description"]
-  labels              = each.value["labels"]
-  clustering          = each.value["clustering"]
-  expiration_time     = each.value["expiration_time"] != null ? each.value["expiration_time"] : 0
-  max_staleness       = each.value["max_staleness"]
-  project             = var.project_id
-  deletion_protection = false
+  for_each                 = local.materialized_views
+  dataset_id               = google_bigquery_dataset.main.dataset_id
+  friendly_name            = each.key
+  table_id                 = each.key
+  description              = each.value["description"]
+  labels                   = each.value["labels"]
+  clustering               = each.value["clustering"]
+  expiration_time          = each.value["expiration_time"] != null ? each.value["expiration_time"] : 0
+  max_staleness            = each.value["max_staleness"]
+  project                  = var.project_id
+  deletion_protection      = false
+  require_partition_filter = each.value["require_partition_filter"]
 
   dynamic "time_partitioning" {
     for_each = each.value["time_partitioning"] != null ? [each.value["time_partitioning"]] : []
     content {
-      type                     = time_partitioning.value["type"]
-      expiration_ms            = time_partitioning.value["expiration_ms"] != null ? time_partitioning.value["expiration_ms"] : 0
-      field                    = time_partitioning.value["field"]
-      require_partition_filter = time_partitioning.value["require_partition_filter"]
+      type          = time_partitioning.value["type"]
+      expiration_ms = time_partitioning.value["expiration_ms"] != null ? time_partitioning.value["expiration_ms"] : 0
+      field         = time_partitioning.value["field"]
     }
   }
 

metadata.yaml

@@ -149,11 +149,11 @@ spec:
               enable_refresh      = bool,
               refresh_interval_ms = string,
               clustering          = list(string),
+              require_partition_filter = optional(bool),
               time_partitioning = object({
                 expiration_ms            = string,
                 field                    = string,
                 type                     = string,
-                require_partition_filter = bool,
               }),
               range_partitioning = object({
                 field = string,

variables.tf

@@ -145,17 +145,17 @@ variable "materialized_views" {
   description = "A list of objects which includes view_id, view_query, clustering, time_partitioning, range_partitioning, expiration_time and labels"
   default     = []
   type = list(object({
-    view_id             = string,
-    description         = optional(string),
-    query               = string,
-    enable_refresh      = bool,
-    refresh_interval_ms = string,
-    clustering          = list(string),
+    view_id                  = string,
+    description              = optional(string),
+    query                    = string,
+    enable_refresh           = bool,
+    refresh_interval_ms      = string,
+    clustering               = list(string),
+    require_partition_filter = optional(bool),
     time_partitioning = object({
-      expiration_ms            = string,
-      field                    = string,
-      type                     = string,
-      require_partition_filter = bool,
+      expiration_ms = string,
+      field         = string,
+      type          = string,
     }),
     range_partitioning = object({
       field = string,