rusher opened a new pull request, #66711: URL: https://github.com/apache/airflow/pull/66711
This PR adds MariaDB compatibility to Airflow, using the existing MySQL backend. this follow https://github.com/apache/airflow/pull/60133, correcting Copilot review ### What's included - **Compatibility fixes**: about 51 lines change [575389bfa979a1208d1ccd724cb6aed31ae9f29d](https://github.com/apache/airflow/commit/575389bfa979a1208d1ccd724cb6aed31ae9f29d) - Migration `0036`: wrap subquery in derived table to avoid MariaDB error 1093. - Migration `0049`: branch on `dialect.is_mariadb` for the 3-argument PCRE2 `REGEXP_REPLACE` signature. - `with_row_locks`: drop the incorrect `supports_for_update_of` guard (MariaDB 10.6+ supports `FOR UPDATE … NOWAIT / SKIP LOCKED` natively). - `_orphan_unreferenced_assets`: switch `.cte()` → `.subquery()` for the `WITH … DELETE` pattern MariaDB's parser rejects pre-12.3. - `test_exceptions`: tolerate driver/server format variants in assertions. in order to permit testing mariadb without testing each commit (https://github.com/apache/airflow/commit/1118fc6a3a84280cbe860a0e9e411d08f75b01f2): - **Breeze support**: new `--mysql-flavor` and `--mariadb-version` options so the same backend code path can target either MySQL or MariaDB locally. - **Weekly canary CI**: scheduled workflow that builds the CI image and runs `breeze testing core-tests --backend mysql --mysql-flavor mariadb --run-db-tests-only --run-in-parallel`. Failures are informational, not a PR gate. canary test works on on fork https://github.com/rusher/airflow/actions/runs/25660511634/job/75319570502 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
