You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/04/11 08:36:28 UTC

[GitHub] [airflow] mik-laj commented on a change in pull request #15301: Remove python2 related handlings and dependencies

mik-laj commented on a change in pull request #15301:
URL: https://github.com/apache/airflow/pull/15301#discussion_r611152675



##########
File path: Dockerfile.ci
##########
@@ -123,7 +121,6 @@ ARG RUNTIME_APT_DEPS="\
       krb5-user \
       ldap-utils \
       less \
-      libpython2.7-stdlib \

Review comment:
       We need to support this case as this is the recommended migration approach for some users who have code that requires libraries only supported by Python 2.7.  Airflow is a task orchestration tool and we dropped Python 2.7 support in the core, but we should still support task orchestrations that use Python 2.7. I think we can rethink this issue when we drop support for Airflow 1.10, because then we can assume that all users have migrated to Airflow 2.0 and all users use Python 3+ as the main version. 




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org