potiuk commented on PR #68511: URL: https://github.com/apache/airflow/pull/68511#issuecomment-5173304045
All review threads are addressed and resolved. The mechanism changed rather than being patched: the prebuilt allow-list is gone in favour of resolving the stored name against `sys.modules`, which fixes the pre-3.2.0 `airflow.exceptions.*` spelling that a `__module__`-keyed map silently dropped — that would have stranded every blob written by 3.0/3.1 on upgrade. It also removes the staleness problem for lazily-imported plugin subclasses, and the `BASE_EXC_SER` comment and message no longer claim a guarantee the encode branch does not give. This one has never had an approval; a look from any of you would let it land for 3.3.1. @kaxil @amoghrajesh @uranusjr --- Drafted-by: Claude Code (Opus 5); reviewed by @potiuk before posting -- 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]
