terraform-provider-databricks
terraform-provider-databricks copied to clipboard
Fix sql table column type upper and lower mismatch error
Changes
Terraform manage column type by lower case so if add the Upper case terraform Terraform manages column types in lowercase, so passing them in uppercase results in a column mismatch error.Even if not changing column type
fix this error https://github.com/databricks/terraform-provider-databricks/issues/3490
Tests
- [x]
make testrun locally - [ ] relevant change in
docs/folder - [ ] covered with integration tests in
internal/acceptance - [x] relevant acceptance tests are passing
- [x] using Go SDK
Hi @case-k-git, thanks for creating the PR, there were other issues that needed to be fixed (mapping) and I have created the PR to include this along with integration test: https://github.com/databricks/terraform-provider-databricks/pull/3528.
Once that is merged, can you please rebase and your PR with unit test can be merged after it.
Hi @tanmay-db Thank you for your comment. Sure!!
Hi @tanmay-db Resolve the conflict. Could you please check when you have time? Thank you.
--- PASS: TestResourceSqlTableUpdateTable_ColumnsTypeUpperLowerCaseThrowsError (0.54s)
PASS
ok github.com/databricks/terraform-provider-databricks/catalog (cached)
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 81.64%. Comparing base (
67ab932) to head (9d0ef1d).
Additional details and impacted files
@@ Coverage Diff @@
## main #3501 +/- ##
=======================================
Coverage 81.64% 81.64%
=======================================
Files 193 193
Lines 19563 19563
=======================================
Hits 15973 15973
Misses 2637 2637
Partials 953 953
Hi @case-k-git, thanks for the PR.