fix: added default value for use_legacy_sql field

Author: pawan1210

README.md

@@ -205,7 +205,7 @@ This module provisions a dataset and a list of tables with associated JSON schem
 | routines | A list of objects which include routine\_id, routine\_type, routine\_language, definition\_body, return\_type, routine\_description and arguments. | <pre>list(object({<br>    routine_id      = string,<br>    routine_type    = string,<br>    language        = string,<br>    definition_body = string,<br>    return_type     = string,<br>    description     = string,<br>    arguments = optional(list(object({<br>      name          = string,<br>      data_type     = string,<br>      argument_kind = string,<br>      mode          = string,<br>    })), []),<br>  }))</pre> | `[]` | no |
 | storage\_billing\_model | Specifies the storage billing model for the dataset. Set this flag value to LOGICAL to use logical bytes for storage billing, or to PHYSICAL to use physical bytes instead. LOGICAL is the default if this flag isn't specified. | `string` | `null` | no |
 | tables | A list of objects which include table\_id, table\_name, schema, clustering, time\_partitioning, range\_partitioning, expiration\_time and labels. | <pre>list(object({<br>    table_id                 = string,<br>    description              = optional(string),<br>    table_name               = optional(string),<br>    schema                   = string,<br>    clustering               = optional(list(string), []),<br>    require_partition_filter = optional(bool),<br>    time_partitioning = optional(object({<br>      expiration_ms = string,<br>      field         = string,<br>      type          = string,<br>    }), null),<br>    range_partitioning = optional(object({<br>      field = string,<br>      range = object({<br>        start    = string,<br>        end      = string,<br>        interval = string,<br>      }),<br>    }), null),<br>    expiration_time     = optional(string, null),<br>    deletion_protection = optional(bool),<br>    labels              = optional(map(string), {}),<br>  }))</pre> | `[]` | no |
-| views | A list of objects which include view\_id and view query | <pre>list(object({<br>    view_id        = string,<br>    description    = optional(string),<br>    query          = string,<br>    use_legacy_sql = bool,<br>    labels         = optional(map(string), {}),<br>  }))</pre> | `[]` | no |
+| views | A list of objects which include view\_id and view query | <pre>list(object({<br>    view_id        = string,<br>    description    = optional(string),<br>    query          = string,<br>    use_legacy_sql = optional(bool, true),<br>    labels         = optional(map(string), {}),<br>  }))</pre> | `[]` | no |
 
 ## Outputs
 

metadata.yaml

@@ -141,7 +141,7 @@ spec:
               view_id        = string,
               description    = optional(string),
               query          = string,
-              use_legacy_sql = bool,
+              use_legacy_sql = optional(bool, true),
               labels         = optional(map(string), {}),
             }))
         defaultValue: []
@@ -397,4 +397,4 @@ spec:
       - iam.googleapis.com
     providerVersions:
       - source: hashicorp/google
-        version: ">= 5.39, < 7"
+        version: ">= 5.39, < 8"

variables.tf

@@ -153,7 +153,7 @@ variable "views" {
     view_id        = string,
     description    = optional(string),
     query          = string,
-    use_legacy_sql = bool,
+    use_legacy_sql = optional(bool, true),
     labels         = optional(map(string), {}),
   }))
 }