Skip to content

Commit

Permalink
Limit ydb to < 3.19.0 (apache#47105)
Browse files Browse the repository at this point in the history
Ydb 3.19.0 wrongly states it compatibility with grpc.io and it
caused our tests to fail as it conflicts with apache.beam < 1.66.

Temporarily limiting it now, deferring final solution(s) to:

* apache/beam#34081

or

* https://github.com/ydb-platform/ydb/issues/15087
  • Loading branch information
potiuk authored and ambika-garg committed Feb 27, 2025
1 parent f64a851 commit c66a149
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 7 deletions.
2 changes: 1 addition & 1 deletion generated/provider_dependencies.json
Original file line number Diff line number Diff line change
Expand Up @@ -1392,7 +1392,7 @@
"apache-airflow-providers-common-sql>=1.20.0",
"apache-airflow>=2.9.0",
"ydb-dbapi>=0.1.0",
"ydb>=3.18.8"
"ydb>=3.18.8, !=3.19.0"
],
"devel-deps": [],
"plugins": [],
Expand Down
8 changes: 4 additions & 4 deletions providers/ydb/README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -50,14 +50,14 @@ The package supports the following python versions: 3.9,3.10,3.11,3.12
Requirements
------------

======================================= ==================
======================================= =====================
PIP package Version required
======================================= ==================
======================================= =====================
``apache-airflow`` ``>=2.9.0``
``apache-airflow-providers-common-sql`` ``>=1.20.0``
``ydb`` ``>=3.18.8``
``ydb`` ``>=3.18.8,!=3.19.0``
``ydb-dbapi`` ``>=0.1.0``
======================================= ==================
======================================= =====================

Cross provider package dependencies
-----------------------------------
Expand Down
8 changes: 7 additions & 1 deletion providers/ydb/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,13 @@ requires-python = "~=3.9"
dependencies = [
"apache-airflow>=2.9.0",
"apache-airflow-providers-common-sql>=1.20.0",
"ydb>=3.18.8",
# Ydb 3.19.0 depends on grpcio >= 1.68.1 but latest apache-beam depends on grpcio < 1.66.0
# also while Ydb requires, it, it does not state it in their requirements, so we cannot add
# grpcio < 1.66 because it will not limit Ydb in any way
# Since using latest apache.beam is more important than latest ydb, we are limiting it until
# the issue https://github.com/apache/beam/issues/34081 or
# https://github.com/ydb-platform/ydb/issues/15087 are fixed
"ydb>=3.18.8, !=3.19.0",
"ydb-dbapi>=0.1.0",
]

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ def get_provider_info():
"dependencies": [
"apache-airflow>=2.9.0",
"apache-airflow-providers-common-sql>=1.20.0",
"ydb>=3.18.8",
"ydb>=3.18.8, !=3.19.0",
"ydb-dbapi>=0.1.0",
],
}

0 comments on commit c66a149

Please sign in to comment.