[PR] [v3-1-test] Fix scheduler crash with email notifications (#56429) [airflow]

["github-actions[bot] (via GitHub)" <gi...@apache.org>]

github-actions[bot] opened a new pull request, #56431:
URL: https://github.com/apache/airflow/pull/56431

   The ``EmailNotificationRequest`` class name (25 characters) exceeded the
   database constraint for ``DbCallbackRequest.callback_type column`` (20
   characters), causing scheduler crashes when email notifications were
   triggered for task failures or retries.
   
   This fix renames the class to ``EmailRequest`` (12 characters) to fit within
   the constraint. A backwards compatibility alias ensures existing DB
   entries with `'EmailNotificationRequest'` can still be deserialized via
   getattr lookup.
   
   The 20-character limit is arbitrary and does not affect performance.
   In a follow-up PR for 3.2, we should increase this to 50+ characters
   to accommodate descriptive class names without requiring abbreviations.
   
   Fixes #56426
   (cherry picked from commit a18fc01dbda319d6670cfab9071b2760a7fc9fe3)
   
   Co-authored-by: Kaxil Naik <ka...@gmail.com>


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [v3-1-test] Fix scheduler crash with email notifications (#56429) [airflow]

["kaxil (via GitHub)" <gi...@apache.org>]

kaxil merged PR #56431:
URL: https://github.com/apache/airflow/pull/56431


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org