Surface table_id in databricks_sql_table (#5058)

## Changes
Surfacing `table_id` in the `databricks_sql_table` resource. The [API
already supports
this](https://docs.databricks.com/api/workspace/tables/get#table_id), so
this change is to surface it in Terraform so other resources can
leverage it.

The change is to add TableID into
[SqlTableInfo](https://github.com/databricks/terraform-provider-databricks/blob/main/catalog/resource_sql_table.go#L47).
When reading the resource, the
[getTable](https://github.com/databricks/terraform-provider-databricks/blob/main/catalog/resource_sql_table.go#L106)
function is called to [fetch from the Get UC Table
API](https://docs.databricks.com/api/workspace/tables/get#table_id),
which should have the table_id field and will therefore be [copied to
the
tableInfo](https://github.com/databricks/terraform-provider-databricks/blob/main/catalog/resource_sql_table.go#L107).

## Tests
`go test -v ./catalog -run "TestResourceSqlTable"` passes.

---------

Co-authored-by: Miles Yucht <[email protected]>

Author: Oscarbachtiar759

NEXT_CHANGELOG.md

@@ -6,6 +6,8 @@
 
 ### New Features and Improvements
 
+ * Add `table_id` to the output of `databricks_sql_table` resource ([#5058](https://github.com/databricks/terraform-provider-databricks/pull/5058)).
+
 ### Bug Fixes
 
 * Allow update `managed_services_customer_managed_key_id` in `databricks_mws_workspaces` ([#5034](https://github.com/databricks/terraform-provider-databricks/pull/5034))

catalog/resource_sql_table.go

@@ -64,6 +64,7 @@ type SqlTableInfo struct {
 	ClusterID           string            `json:"cluster_id,omitempty" tf:"computed"`
 	WarehouseID         string            `json:"warehouse_id,omitempty"`
 	Owner               string            `json:"owner,omitempty" tf:"computed"`
+	TableID             string            `json:"table_id" tf:"computed"`
 
 	exec    common.CommandExecutor
 	sqlExec sql.StatementExecutionInterface

catalog/resource_sql_table_test.go

@@ -1779,6 +1779,7 @@ func TestResourceSqlTable_Diff_ExistingResource(t *testing.T) {
 			"",
 			map[string]string{
 				"effective_properties.%": "0",
+				"table_id":               "test-table-id-123",
 			},
 			nil,
 		},
@@ -1789,6 +1790,7 @@ func TestResourceSqlTable_Diff_ExistingResource(t *testing.T) {
 				"effective_properties.%":                   "2",
 				"effective_properties.computedprop":        "computedvalue",
 				"effective_properties.option.computedprop": "computedvalue",
+				"table_id": "test-table-id-123",
 			},
 			nil,
 		},
@@ -1811,6 +1813,7 @@ func TestResourceSqlTable_Diff_ExistingResource(t *testing.T) {
 				"effective_properties.myprop":       "myval",
 				"effective_properties.option.myopt": "myval",
 				"effective_properties.spark.sql.dataSourceOptions.multiLine": "true",
+				"table_id": "test-table-id-123",
 			},
 			nil,
 		},
@@ -1832,6 +1835,7 @@ func TestResourceSqlTable_Diff_ExistingResource(t *testing.T) {
 				"effective_properties.computedprop":       "computedval",
 				"effective_properties.option.myopt":       "myval",
 				"effective_properties.option.computedopt": "computedval",
+				"table_id":                                "test-table-id-123",
 			},
 			nil,
 		},
@@ -1852,6 +1856,7 @@ func TestResourceSqlTable_Diff_ExistingResource(t *testing.T) {
 				"effective_properties.%":            "2",
 				"effective_properties.myprop":       "otherval",
 				"effective_properties.option.myopt": "otherval",
+				"table_id":                          "test-table-id-123",
 			},
 			map[string]*terraform.ResourceAttrDiff{
 				"effective_properties.myprop":       {New: "myval", Old: "otherval"},

docs/data-sources/table.md

@@ -45,6 +45,8 @@ This data source exports the following attributes:
   * `columns` - Array of ColumnInfo objects of the table's columns
   * `owner` - Current owner of the table
   * `comment` - Free-form text description
+  * `table_id` - The unique identifier of the table.
+
 
 ## Related Resources
 

docs/resources/sql_table.md

@@ -204,6 +204,7 @@ Currently, changing the column definitions for a table will require dropping and
 In addition to all the arguments above, the following attributes are exported:
 
 * `id` - ID of this table in the form of `<catalog_name>.<schema_name>.<name>`.
+* `table_id` - The unique identifier of the table.
 
 ## Import