Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhanced Support for Airflow Callbacks #222

Closed

Conversation

jroach-astronomer
Copy link
Member

@jroach-astronomer jroach-astronomer commented Sep 20, 2024

This is only a draft PR. The goal of this work is to provide enhanced support for callbacks, per the request in #209. Currently, only a fully-defined callback can be passed as a value to any callback key. This PR provides support for callbacks to be defined in a manner similar to Python callables.

@jroach-astronomer
Copy link
Member Author

@pankajastro, this PR is NOT yet ready to be merged. But do you mind taking a quick peek. This allows for the functionality mentioned in Issue 209.

@tatiana tatiana marked this pull request as draft September 21, 2024 09:34
@pankajastro
Copy link
Contributor

Hi @jroach-astronomer, could you please add a description and update the PR title? This will help me better understand the fixes.

@jroach-astronomer jroach-astronomer changed the title Issue 209 Enhanced Support for Airflow Callbacks Sep 24, 2024
@jroach-astronomer
Copy link
Member Author

@pankajastro, just wanted to follow up on this. Thanks!

@pankajkoti
Copy link
Contributor

hi @jroach-astronomer , the CI seems to be failing and there are some conflicts. Would you be able to address those, please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants