[Feature] using empty string as default for identity column

databricks · Aug 19, 2024 · cc08c2e · cc08c2e
1 parent d5c036a
commit cc08c2e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/catalog/resource_sql_table.go b/catalog/resource_sql_table.go
@@ -24,7 +24,7 @@ var MaxSqlExecWaitTimeout = 50
 type SqlColumnInfo struct {
 	Name     string         `json:"name"`
 	Type     string         `json:"type_text,omitempty" tf:"alias:type,computed"`
-	Identity IdentityColumn `json:"identity,omitempty" tf:"default:false"`
+	Identity IdentityColumn `json:"identity,omitempty"`
 	Comment  string         `json:"comment,omitempty"`
 	Nullable bool           `json:"nullable,omitempty" tf:"default:true"`
 }

diff --git a/docs/resources/sql_table.md b/docs/resources/sql_table.md
@@ -177,7 +177,7 @@ Currently, changing the column definitions for a table will require dropping and
 
 * `name` - User-visible name of column
 * `type` - Column type spec (with metadata) as SQL text. Not supported for `VIEW` table_type.
-* `identity` - (Optional) Whether field is an identity column. Can be `default`, `always` or `false`. (Default: `false`)
+* `identity` - (Optional) Whether field is an identity column. Can be `default`, `always` or ``. (Default: ``)
 * `comment` - (Optional) User-supplied free-form text.
 * `nullable` - (Optional) Whether field is nullable (Default: `true`)