potiuk commented on PR #39606:
URL: https://github.com/apache/airflow/pull/39606#issuecomment-2123410713

   > @potiuk Airflow 2.8.4 requires (constraints) universal-path 0.1.4 which is 
indeed not compatible with python 3.12. However., I am not sure if this is 
related. The `namespace` property was added past in 2.9.0, so I guess the 
provider is only compatible with Airflow 2.9.0+ without adjustments?
   
   Yes. That was my thought as well (and that's how I solved it in the PR) - In 
this case I check if we are retrieving openlineage facets in Airlfow < 2.9.0 in 
this operator and fail it explicitly if so:  
https://github.com/apache/airflow/pull/39606/files#diff-ee30b541cc8033613196b62a0e5d9ff0276113f27d011a9ffbcd355826205f14R87
  . Note that FileTransferOperator should work **fine** when openlineage is 
disabled even in 2.8.*
   
   Not sure if this is the best approach @mobuchowski @kacpermuda -> maybe we 
can make it works also in 2.8 ? 
   


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