Skip to content

Commit 61fb902

Browse files
authored
[Internal] Fix Quality Monitor test by removing parallelism (#4568)
## 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`. This is already fixed in its non-plugin framework counterpart. #4566 ## 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 6114f2a commit 61fb902

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

internal/providers/pluginfw/products/qualitymonitor/resource_quality_monitor_acc_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"
@@ -100,6 +101,7 @@ func TestUcAccQualityMonitor(t *testing.T) {
100101
}
101102
102103
resource "databricks_sql_table" "mySnapshot" {
104+
depends_on = [databricks_quality_monitor.testMonitorTimeseries]
103105
catalog_name = databricks_catalog.sandbox.id
104106
schema_name = databricks_schema.things.name
105107
name = "bar{var.STICKY_RANDOM}_snapshot"

0 commit comments

Comments
 (0)