1fanwang commented on PR #66787: URL: https://github.com/apache/airflow/pull/66787#issuecomment-4445493556
Closing in favour of #66888. After more thought, the original proposal here adds a config knob, a new exception class, and per-route validation for what is fundamentally a "DB rejected the payload" failure that the FastAPI exception-handler layer can translate for free across every endpoint. The new PR ships exactly one handler for `sqlalchemy.exc.DataError`, registers it on both the public REST API and the execution API, and inherits the translation on every existing and future write endpoint (DagRun `conf`, Connection `extra`, Variable `val`, XCom `value`, TaskInstance `note`, HITL fields, etc) with no new configuration surface. Leaving this PR open as reference for the trade-off conversation. --- Drafted-by: Claude Code (Opus 4.7) (no human review 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]
