The GitHub Actions job "Tests (AMD)" on airflow.git/parameterize-hive-stats-sql has succeeded. Run started by GitHub user Har1sh-k (triggered by potiuk).
Head commit for run: 62943d0e1a97737ee343d71a7530fb4b1463d175 / Harish Kolla <[email protected]> Split the Presto table name only on dots outside quoted identifiers Quoting the table name per `self.table.split(".")` component treated a dot inside a quoted identifier as a catalog/schema separator, so each fragment matched neither the plain-identifier nor the quoted-identifier regex and both halves were re-escaped: `"my.table"` became `"""my"."table"""` and `db."odd.name"` became `db."""odd"."name"""`. Plain `db.tbl` and bare `tbl` were unaffected, so this only hit the quoted-with-dot case, which is exactly the case the quoting support exists for. Move the assembly into `_quote_presto_table`, which splits on the dots *outside* a quoted identifier, leaving a quoted component that contains a dot intact. Add `test_quote_presto_table` pinning the pass-through behaviour (bare, qualified-plain, special-character, pre-quoted, embedded doubled quote, and quoted-containing-a-dot), so a future refactor cannot quietly reintroduce the double-escaping, plus the dotted cases at the emitted-SQL level in `test_execute_quotes_table_identifier`. Report URL: https://github.com/apache/airflow/actions/runs/32202010251 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
