The GitHub Actions job "Tests (AMD)" on airflow.git/tidb-asset-collation-config 
has succeeded.
Run started by GitHub user 1fanwang (triggered by 1fanwang).

Head commit for run:
850c074942cc5a1c2e98f189f75ccfca4e3c3d78 / 1fanwang <[email protected]>
Apply the asset collation setting to migrations as well as models

The asset name/uri/group columns carry an explicit MySQL collation so their
1500-character unique indexes stay inside the 3072-byte index limit. That
collation is configurable on the models, but the migrations that create and
alter those columns still hard-coded `latin1_general_cs`, so the setting only
took effect when the schema was created from the ORM.

A fresh install on a MySQL-compatible engine that lacks that collation replays
the migrations instead and fails on the first asset table, which leaves the
setting useless in exactly the case it was added for.

Resolve the collation at migration run time through the same configuration key
the models read.

Signed-off-by: 1fanwang <[email protected]>

Report URL: https://github.com/apache/airflow/actions/runs/31211053299

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to