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

Create SqlToSlackOperator #24243

Closed
1 task done
eladkal opened this issue Jun 6, 2022 · 2 comments · Fixed by #24663
Closed
1 task done

Create SqlToSlackOperator #24243

eladkal opened this issue Jun 6, 2022 · 2 comments · Fixed by #24663

Comments

@eladkal
Copy link
Contributor

eladkal commented Jun 6, 2022

Body

Currently we have SnowflakeToSlackOperator & PrestoToSlackOperator.
This task is to create a generic SqlToSlackOperator to be compatible with all hooks that has get_pandas_df function.
The new operator should be in providers/slack/transfers/sql_to_slack.py, once completed we can also deprecate SnowflakeToSlackOperator & PrestoToSlackOperator.

A good resource for reference can be SqlToS3Operator which the first part of the operator (running query and creating dataframe should be identical), the 2nd part of creating the slack message should be very similar to what the current operators do.

Committer

  • I acknowledge that I am a maintainer/committer of the Apache Airflow project.
@alexkruc
Copy link
Contributor

alexkruc commented Jun 6, 2022

I'm willing to take this one :)
Hopefully, I could manage my way around this issue :)

@potiuk
Copy link
Member

potiuk commented Jun 6, 2022

assigned you

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

Successfully merging a pull request may close this issue.

3 participants