naruto-lgtm commented on PR #68662: URL: https://github.com/apache/airflow/pull/68662#issuecomment-5101078479
Fixed the upgrade path in a55dd765. The two tags are recognised again on the decode side only: an old `airflow_exc_ser`/`base_exc_ser` row now deserializes to the exception's message instead of raising `TypeError`. Nothing emits them anymore, so no new row is ever written in that format and there's no `import_string` on a payload-supplied name to come back. I kept it to the plain message rather than rendering the class name, so an old row is indistinguishable from a new one: `str(exc)` is what a live exception serializes to now. That also settles the encode-side note I raised, since the old-worker compat converter in `v2026_04_06.py` and the legacy decode agree on the same form. Tests go both ways. Legacy blobs for `AirflowException`, `AirflowRescheduleException` and `KeyError` come back as their message, and one naming `os.system` or `exec` returns the string with nothing executed. All 5 fail on the previous head. -- 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]
