While cherry-picking some of the latest "prod-image" related changes to
v1-10-test I also cherry-picked the 'isort' change - this way it will be a
bit easier to cherry-pick any changes that were done after the isort.

While doing it I found that we had some python3-only imports in some of the
files. Not sure why this worked for py27 tests. It's likely that those
operators were broken in 1.10.6/1.10.5 etc. unless we had some magic python
library piggybacking in place.

I created a JIRA issue for that
https://issues.apache.org/jira/browse/AIRFLOW-5851 .

I fixed all the problems (maybe not in the best way possible - just wanted
to minimise the changes and cross-dependencies). I pushed the changes to
v1-10-test and wait for the tests to pass, but maybe someone might take a
look if the changes look ok. I am happy to correct it.

Here is the commit:
https://github.com/apache/airflow/commit/c26bac0e55b574e5b148d0d91c1756002dfa3bae


Those files were affected:

   - airflow/contrib/operators/gcp_cloud_build_operator.py
   - airflow/models/connection.py
   - airflow/contrib/hooks/databricks_hook.py
   - airflow/contrib/operators/s3_to_sftp_operator.py
   - airflow/contrib/operators/sftp_to_s3_operator.py
   - airflow/sensors/s3_key_sensor.py
   - airflow/utils/log/gcs_task_handler.py


J.

-- 

Jarek Potiuk
Polidea <https://www.polidea.com/> | Principal Software Engineer

M: +48 660 796 129 <+48660796129>
[image: Polidea] <https://www.polidea.com/>

Reply via email to