potiuk commented on code in PR #44607:
URL: https://github.com/apache/airflow/pull/44607#discussion_r1867787349


##########
airflow/__init__.py:
##########
@@ -58,10 +58,20 @@
 # very easily cause import cycles in the conf init/validate code (since 
downstream code from
 # those functions likely import settings).
 # configuration is therefore initted early here, simply by importing it.
+
+from packaging.version import Version
+
 from airflow import configuration, settings
 
+

Review Comment:
   I think - as discussed in slack - this is  a nice proposal. that we might be 
able to use in some 8 months (if we back-port it and release in 2.11, but we 
need to figure out - as part of it - what we do now and how we do it for the 
rest of the code.
   
   I proposed a solution there (this needs to be fixed) - 
https://github.com/apache/airflow/pull/43773#discussion_r1867669213 - not sure 
if we can figure out something better. I think it's a good idea to discuss it 
there.



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