Brunda10 commented on PR #55408:
URL: https://github.com/apache/airflow/pull/55408#issuecomment-3273517545

   Thanks for the feedback @ashb 
   Sorry for the confusion , removing extra directly would be a breaking change 
for DAGs that still use it.
   
   In the issue discussion it was suggested to deprecate Asset.extra (to be 
removed in 4.0) and introduce event_extra_template as the clearer replacement. 
To align with that, my plan would be to:
   
   - keep extra in 3.x,
   
   - raise a DeprecationWarning when it’s used,
   
   - and document the migration path to event_extra_template.
   
   Just to clarify: this change is only in the Asset definition. The DB model 
still has the extra column unchanged for now.
   
   Please let me know if this approach is okay, or if you prefer I support both 
fully without deprecation in 3.x.
   


-- 
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]

Reply via email to