Skip to content

Commit

Permalink
Fixed databricks_table updates (#1187)
Browse files Browse the repository at this point in the history
Fix #1175
  • Loading branch information
nfx committed Mar 14, 2022
1 parent 9ce2e6b commit e67df6a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
* Use correct HTTP verb for modifying `databricks_permissions` on `databricks_sql_endpoint` entities. Authorized user, assumingly part of `admins` group, is no longer sending `CAN_MANAGE` permission in the HTTP PUT request ([#1163](https://github.com/databrickslabs/terraform-provider-databricks/issues/1163)).
* Added diff suppression for `min_num_clusters` field in `databricks_sql_endpoint` ([#1172](https://github.com/databrickslabs/terraform-provider-databricks/pull/1172)).
* Added special case for handling `Cannot access cluster that was terminated or unpinned more than 30 days ago` error in `databricks_cluster` as an indication of resource removed on the platform side ([#1177](https://github.com/databrickslabs/terraform-provider-databricks/issues/1177)).
* Fixed updating of `databricks_table` resources ([#1175](https://github.com/databrickslabs/terraform-provider-databricks/issues/1175)).

Updated dependency versions:

Expand Down
2 changes: 1 addition & 1 deletion catalog/resource_table.go
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ func (a TablesAPI) getTable(name string) (ti TableInfo, err error) {
}

func (a TablesAPI) updateTable(ti TableInfo) error {
return a.client.Patch(a.context, "/unity-catalog/tables/"+ti.Name, ti)
return a.client.Patch(a.context, "/unity-catalog/tables/"+ti.FullName(), ti)
}

func (a TablesAPI) deleteTable(name string) error {
Expand Down

0 comments on commit e67df6a

Please sign in to comment.