Make `TableName` field part of quality monitor schema

pietern commented 1 week ago

Changes

This field was special-cased in #1307 because it's not part of the JSON payload in the SDK struct.

This approach, while pragmatic, meant it didn't show up in the JSON schema. While debugging an issue with quality monitors in #1900, I couldn't figure out why I was getting schema errors on this field, or how it was passed through to the TF representation. This commit removes the special case and makes it behave like everything else.

Tests

Unit tests pass.
Confirmed that the updated schema failed validation before this change.

github-actions[bot] commented 1 week ago

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger: go/deco-tests-run/cli

Inputs:

PR number: 1903
Commit SHA: 73263c923f31f47dd326d54a5334665eeb68dca2

Checks will be approved automatically on success.

eng-dev-ecosystem-bot commented 1 week ago

Test Details: go/deco-tests/11840837328

databricks / cli

Make `TableName` field part of quality monitor schema #1903

Changes

Tests