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

   It's just a matter of looking it up in git blame.
   
   Actually this is not really good idea for now (Provider's release manager's 
releasing providers should not use Python 3.12). it's all visible in detail in 
history of those changed lines.
   
   What can **actually** be removed now is the 3.9 check (because we do not use 
Python 3.8 any more and python 3.8 produced non-reproducible builds as 
explained in comments and in https://github.com/apache/airflow/pull/37401). \
   
   Python 3.12 was excluded (additionally) here 
https://github.com/apache/airflow/pull/37615 because some of our provider 
packages did not support 3.12 and as of recently as part of release process we 
are building .sdist packages and they cannot be build for packages that do not 
support Python 3.12. Currently apache-beam does not support Python 3.12 yet 
(mostly because they embedded dill version that breaks our 
PythonVirtualenvOperator as explained in 
https://github.com/apache/airflow/pull/41990 and we cannot enabvle it until 
https://github.com/apache/beam/issues/32617 is fixed.
   
   But yeah. It makes no sense to  block 3.12 usage for everyone, it might be 
enough to remove it for providers release command only. I will correct it.


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