nagasrisai commented on PR #69959: URL: https://github.com/apache/airflow/pull/69959#issuecomment-5181295451
Thanks for the pointers! I've pushed two follow-up commits: 1. Narrowed the `except IntegrityError` guard to unique-constraint violations only (matching the per-dialect strings in `_UniqueConstraintErrorHandler`) so FK violations propagate correctly instead of being silently swallowed. 2. Added two tests to `TestXComsSetEndpoint` — one that exercises the UPDATE fallback path by patching `XComModel.set` to raise a unique-constraint error, and one that confirms FK errors propagate as a 500. Will also look into getting the static checks clean. -- 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]
