Skip to content

Pin databricks-sql-connector>=4.0.0 in generated constraints#69863

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:constraint-databricks-sql-connector
Jul 14, 2026
Merged

Pin databricks-sql-connector>=4.0.0 in generated constraints#69863
potiuk merged 1 commit into
apache:mainfrom
potiuk:constraint-databricks-sql-connector

Conversation

@potiuk

@potiuk potiuk commented Jul 14, 2026

Copy link
Copy Markdown
Member

During highest-resolution constraint generation, the resolver preferred the newest thrift (0.24.0, from apache/thrift#3584) over the newest databricks-sql-connector. Because current databricks-sql-connector requires thrift<=0.23.0, the resolver downgraded the connector — which in turn dragged the databricks provider back to an old version in the PyPI constraints.

Pinning databricks-sql-connector>=4.0.0 in additional_constraints_for_highest_resolution keeps the constraints installable with a current connector and provider.

Tracked upstream at databricks/databricks-sql-python#859.

closes: #69603


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 4.8, 1M context)

Generated-by: Claude Code (Opus 4.8, 1M context) following the guidelines

thrift 0.24.0 (apache/thrift#3584) is incompatible with the
databricks-sql-connector thrift<=0.23.0 requirement, so during highest
resolution the resolver preferred the newest thrift and downgraded the
connector, which in turn dragged the databricks provider back to an old
version. Pinning the connector to >=4.0.0 keeps PyPI constraints
installable with a current databricks-sql-connector and provider.

closes: apache#69603
@potiuk

potiuk commented Jul 14, 2026

Copy link
Copy Markdown
Member Author

FYI: @vatsrahul1001 -> this one is important for 3.3.1 to merge as the databricks provider got downgraded because of interaction between thrift and databricks-mysql-connector dependencies.

@eladkal - I will also separately add protection to flag when things like that happen in the future - I think downgrading any of our providers is a sign that something weird happens with dependency resolution.

@eladkal

eladkal commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

@eladkal - I will also separately add protection to flag when things like that happen in the future - I think downgrading any of our providers is a sign that something weird happens with dependency resolution

Flag how? The mechanism doesn't involve release manager. Are you thinking about Slack notification? Or a manual step during core release that will verify the constraints file against this issue?

@potiuk

potiuk commented Jul 14, 2026

Copy link
Copy Markdown
Member Author

Before the release -> posting message to internal-airflow-ci-cd when provider gets downgraded . Those things do not happen at release time - this one happened literally few days before

See: #69870

@potiuk potiuk merged commit 98b4b01 into apache:main Jul 14, 2026
150 checks passed
@potiuk potiuk deleted the constraint-databricks-sql-connector branch July 14, 2026 12:22
@github-actions

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-3-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
v3-3-test PR Link

github-actions Bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jul 14, 2026
…nts (apache#69863)

thrift 0.24.0 (apache/thrift#3584) is incompatible with the
databricks-sql-connector thrift<=0.23.0 requirement, so during highest
resolution the resolver preferred the newest thrift and downgraded the
connector, which in turn dragged the databricks provider back to an old
version. Pinning the connector to >=4.0.0 keeps PyPI constraints
installable with a current databricks-sql-connector and provider.
(cherry picked from commit 98b4b01)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
closes: apache#69603
aws-airflow-bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jul 14, 2026
…nts (apache#69863)

thrift 0.24.0 (apache/thrift#3584) is incompatible with the
databricks-sql-connector thrift<=0.23.0 requirement, so during highest
resolution the resolver preferred the newest thrift and downgraded the
connector, which in turn dragged the databricks provider back to an old
version. Pinning the connector to >=4.0.0 keeps PyPI constraints
installable with a current databricks-sql-connector and provider.
(cherry picked from commit 98b4b01)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
closes: apache#69603
eladkal pushed a commit that referenced this pull request Jul 14, 2026
…nts (#69863) (#69871)

thrift 0.24.0 (apache/thrift#3584) is incompatible with the
databricks-sql-connector thrift<=0.23.0 requirement, so during highest
resolution the resolver preferred the newest thrift and downgraded the
connector, which in turn dragged the databricks provider back to an old
version. Pinning the connector to >=4.0.0 keeps PyPI constraints
installable with a current databricks-sql-connector and provider.
(cherry picked from commit 98b4b01)


closes: #69603

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Databricks provider and numpy constraint for python 3.11 and 3.12 for Airflow 3.3.0 got downgraded

3 participants