What this PR does / why we need it:

_bq_ensure_monitoring_tables puts PRIMARY KEY (...) NOT ENFORCED after the closing parenthesis of the column list in all four data quality monitoring CREATE TABLE IF NOT EXISTS statements. GoogleSQL only accepts table constraints inside the table element list (( column | constraint_definition [, ...] )), so BigQuery rejects each statement at parse time:

400 PRIMARY KEY must be defined in the table element list as column attribute or constraint.

As a result, BigQuery monitoring (feast monitor run, the /monitoring REST endpoints, and auto_baseline on feast apply) fails on every BigQuery deployment. This has been the case since v0.64.0.

This PR:

• moves each PRIMARY KEY (...) NOT ENFORCED inside its column list, after a trailing comma on the last column. The key columns are unchanged, and nothing else in the file changes.
• adds test_ensure_monitoring_tables_primary_key_inside_column_list to sdk/python/tests/unit/infra/offline_stores/test_bigquery.py. It patches _get_bigquery_client, captures the DDL sent to client.query, and asserts two things: all four monitoring tables are created, and in each CREATE TABLE the PRIMARY KEY ... NOT ENFORCED clause sits inside the outermost parentheses, with nothing after the final ). Until now, no test covered this function: the monitoring integration test mocks ensure_monitoring_tables.

The other offline stores (Redshift, Snowflake, Postgres, Oracle) already put the constraint inside the element list, so only BigQuery needed the fix.

Which issue(s) this PR fixes:

Fixes #7020

Checks

• I've made sure the tests are passing.
• My commits are signed off (git commit -s)
• My PR title follows conventional commits format

Testing Strategy

• Unit tests
• Integration tests
• Manual tests
• Testing is not required for this change

Unit test, before and after the fix (pytest sdk/python/tests/unit/infra/offline_stores/test_bigquery.py):

• Without the fix: 1 failed, 16 passed. The new test fails because the feast_monitoring_feature_metrics DDL ends with ... NOT ENFORCED instead of ).
• With the fix: 17 passed

Lint: ruff check, ruff format --check (ruff 0.16.10, the pinned CI version) on sdk/python/feast/ and sdk/python/tests/, and mypy on both changed files. All clean.

Manual: a BigQuery dry run confirms the parser behavior, and the result doesn't depend on location:

bq query --dry_run --use_legacy_sql=false \
  'CREATE TABLE IF NOT EXISTS `my-project.my_dataset.t` (job_id STRING NOT NULL, status STRING) PRIMARY KEY (job_id) NOT ENFORCED'
# Error: PRIMARY KEY must be defined in the table element list as column attribute or constraint.

bq query --dry_run --use_legacy_sql=false \
  'CREATE TABLE IF NOT EXISTS `my-project.my_dataset.t` (job_id STRING NOT NULL, status STRING, PRIMARY KEY (job_id) NOT ENFORCED)'
# Query successfully validated.

Misc

🤖 Generated with Claude Code