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/08/23 20:52:55 UTC
[GitHub] [airflow] ashwin153 opened a new issue #17795: Circular Dependency in Apache Airflow 2.1.3
ashwin153 opened a new issue #17795:
URL: https://github.com/apache/airflow/issues/17795
**Apache Airflow version**:
2.1.3
**OS**:
Ubuntu 20.04 LTS
**Deployment**:
Bazel
**What happened**:
When I tried to bump my Bazel monorepo from 2.1.2 to 2.1.3, Bazel complains that there is the following circular dependency.
```
ERROR: /github/home/.cache/bazel/_bazel_bookie/c5c5e4532705a81d38d884f806d2bf84/external/pip/pypi__apache_airflow/BUILD:11:11: in py_library rule @pip//pypi__apache_airflow:pypi__apache_airflow: cycle in dependency graph:
//wager/publish/airflow:airflow
.-> @pip//pypi__apache_airflow:pypi__apache_airflow
| @pip//pypi__apache_airflow_providers_http:pypi__apache_airflow_providers_http
`-- @pip//pypi__apache_airflow:pypi__apache_airflow
```
**What you expected to happen**:
No dependency cycles.
**How to reproduce it**:
A concise reproduction will require some effort. I am hoping that there is a quick resolution to this, but am willing to create a reproduction if it is required to determine the root cause.
**Anything else we need to know**:
Perhaps related to apache/airflow#14128.
**Are you willing to submit a PR?**
Yes
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk edited a comment on issue #17795: Circular Dependency in Apache Airflow 2.1.3
Posted by GitBox <gi...@apache.org>.
potiuk edited a comment on issue #17795:
URL: https://github.com/apache/airflow/issues/17795#issuecomment-904169637
Yeah . HTTP provider has been brought back to be preinstalled and we have to fix that and release it to remove the circular dependency, This will be fixed with the next release of HTTP provider (https://github.com/apache/airflow/pull/17796)
BTW. Bazel is not supported as the way of installing Airflow. The only official way of installing airflow is via `pip`and constraints -https://airflow.apache.org/docs/apache-airflow/stable/installation.html#installation-tools.
PIP can handle circular dependencies well. So it's really Bazel's problem not Airflow (we are happy to fix it anyway for the next release - thanks for reporting) .
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk edited a comment on issue #17795: Circular Dependency in Apache Airflow 2.1.3
Posted by GitBox <gi...@apache.org>.
potiuk edited a comment on issue #17795:
URL: https://github.com/apache/airflow/issues/17795#issuecomment-904169637
Yeah . HTTP provider has been brought back to be preinstalled and we have to fix that and release it to remove the circular dependency, This will be fixed with the next release of HTTP provider (https://github.com/apache/airflow/pull/17796)
BTW. Bazel is not supported as the way of installing Airflow. The only official way of installing airflow is via `pip`and constraints -https://airflow.apache.org/docs/apache-airflow/stable/installation.html#installation-tools.
PIP can handle circular dependencies well. So it's really Bazel's problem not Airflow (we are happy to fix it anyway for the next release - thenks for reporting) .
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on issue #17795: Circular Dependency in Apache Airflow 2.1.3
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #17795:
URL: https://github.com/apache/airflow/issues/17795#issuecomment-904169637
Yeah . HTTP provider has been brought back to be preinstalled and we had to release it to remove the circular dependency, This will be fixed with the next release of HTTP provider (https://github.com/apache/airflow/pull/17796)
BTW. Bazel is not supported as the way of installing Airflow. The only official way of installing airflow is via `pip`and constraints -https://airflow.apache.org/docs/apache-airflow/stable/installation.html#installation-tools.
PIP can handle circular dependencies well. So it's really Bazel's problem not Airflow (we are happy to fix it anyway for the next release - thenks for reporting) .
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk closed issue #17795: Circular Dependency in Apache Airflow 2.1.3
Posted by GitBox <gi...@apache.org>.
potiuk closed issue #17795:
URL: https://github.com/apache/airflow/issues/17795
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] uranusjr commented on issue #17795: Circular Dependency in Apache Airflow 2.1.3
Posted by GitBox <gi...@apache.org>.
uranusjr commented on issue #17795:
URL: https://github.com/apache/airflow/issues/17795#issuecomment-904329243
FWIW circular dependencies are allowed in Python (in other words, a project’s Python package dependencies is not a DAG), so Bazel not supporting it is a bug (or missing feature).
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org