vandonr commented on PR #38155:
URL: https://github.com/apache/airflow/pull/38155#issuecomment-2047252150

   I didn't change the signature of the existing openlinage listener
   
https://github.com/apache/airflow/blob/a2f5307fd0ec54b34b8c753a53024a2629a56fd8/airflow/providers/openlineage/plugins/listener.py#L196
   because pluggy supports having too many parameters, but it crashes when 
calling a method with too few params, so we have to keep the old signature as 
not to break compatibility of newer provider package versions with older 
airflow versions.


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