Skip to content

Conversation

@phi-friday
Copy link
Contributor

Made @task.docker use serializer instead of use_dill.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@phi-friday phi-friday changed the title Feat docker serializer feat(docker): Replace use_dill with serializer Aug 9, 2024
Copy link
Contributor

@shahar1 shahar1 left a comment

Choose a reason for hiding this comment

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

LGTM

@phi-friday phi-friday force-pushed the feat-docker-serializer branch from 08d6c02 to 2ee5681 Compare August 14, 2024 05:38
@phi-friday
Copy link
Contributor Author

error in common.sql
related: #41460

@phi-friday phi-friday force-pushed the feat-docker-serializer branch from 2ee5681 to c7b54a1 Compare August 14, 2024 08:45
@potiuk potiuk merged commit 901f00d into apache:main Aug 14, 2024
@phi-friday phi-friday deleted the feat-docker-serializer branch August 14, 2024 11:23
Artuz37 pushed a commit to Artuz37/airflow that referenced this pull request Aug 19, 2024
romsharon98 pushed a commit to romsharon98/airflow that referenced this pull request Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants