Skip to content

Commit de416c9

Browse files
authored
[Internal] Fix Quality Monitor test by removing parallelism (#4566)
## Changes <!-- Summary of your changes that are easy to understand --> This PR removes parallelism in the quality monitor test. Currently, we create multiple databricks_sql_table concurrently, which causes the following error: `[OCC check failure] Entity proto at /metastores:<metastore_id> may have been updated during the transaction`. ## Tests <!-- How is this tested? Please see the checklist below and also describe any other relevant tests --> - [ ] `make test` run locally - [ ] relevant change in `docs/` folder - [ ] covered with integration tests in `internal/acceptance` - [ ] using Go SDK - [ ] using TF Plugin Framework NO_CHANGELOG=true
1 parent f50e02d commit de416c9

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

catalog/quality_monitor_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@ func TestUcAccQualityMonitor(t *testing.T) {
7575
}
7676
7777
resource "databricks_sql_table" "myTimeseries" {
78+
depends_on = [databricks_quality_monitor.testMonitorInference]
7879
catalog_name = databricks_catalog.sandbox.id
7980
schema_name = databricks_schema.things.name
8081
name = "bar{var.STICKY_RANDOM}_timeseries"
@@ -103,6 +104,7 @@ func TestUcAccQualityMonitor(t *testing.T) {
103104
}
104105
105106
resource "databricks_sql_table" "mySnapshot" {
107+
depends_on = [databricks_quality_monitor.testMonitorTimeseries]
106108
catalog_name = databricks_catalog.sandbox.id
107109
schema_name = databricks_schema.things.name
108110
name = "bar{var.STICKY_RANDOM}_snapshot"

0 commit comments

Comments
 (0)