Skip to content

Conversation

@jscheffl
Copy link
Contributor

As we see CI failing on fastavro with Python 3.13 - see #60543 (comment) - this PR bumps fastavro to be 1.10 at least


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@jscheffl jscheffl added full tests needed We need to run full set of tests for this PR to merge all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs labels Jan 17, 2026
@jscheffl jscheffl marked this pull request as ready for review January 17, 2026 21:14
@potiuk potiuk merged commit e7d7bf1 into apache:main Jan 17, 2026
226 checks passed
@jason810496
Copy link
Member

Thanks Jens!

]
"avro" = [
"fastavro>=1.9.0"
'fastavro>=1.10.0;python_version>="3.12"' # Need to pin to this version for Python 3.13 compatibility
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How come the python version is 3.12 here and 3.13 elsewhere?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right, I should have not lost the previous pinning for all Python versions <3.13. It was mainly added becasuse Python 3.13 needs at least 1.10. But the other should not be replaced.
Might be by accident so far not detected as the optional dependency is not used in CI and only picked if manually selected to install.
Will raise a corrective PR later today

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See #60797

jscheffl added a commit to jscheffl/airflow that referenced this pull request Jan 19, 2026
jscheffl added a commit that referenced this pull request Jan 19, 2026
jason810496 pushed a commit to jason810496/airflow that referenced this pull request Jan 22, 2026
…60732)

* Pin fastavro to minimum 1.10.0 for Python 3.13 compatibility

* Pin fastavro to minimum 1.10.0 for Python 3.13 compatibility
jason810496 pushed a commit to jason810496/airflow that referenced this pull request Jan 22, 2026
suii2210 pushed a commit to suii2210/airflow that referenced this pull request Jan 26, 2026
shreyas-dev pushed a commit to shreyas-dev/airflow that referenced this pull request Jan 29, 2026
…60732)

* Pin fastavro to minimum 1.10.0 for Python 3.13 compatibility

* Pin fastavro to minimum 1.10.0 for Python 3.13 compatibility
shreyas-dev pushed a commit to shreyas-dev/airflow that referenced this pull request Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs area:providers full tests needed We need to run full set of tests for this PR to merge kind:documentation provider:apache-hdfs provider:databricks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants