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.

Sign up for GitHub

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

Jump to bottom

RedshiftDataOperator: Add support for Redshift serverless clusters #32280

Closed
1 of 2 tasks
cw-somil opened this issue Jun 30, 2023 · 4 comments · Fixed by #32785
Closed
1 of 2 tasks

RedshiftDataOperator: Add support for Redshift serverless clusters #32280

cw-somil opened this issue Jun 30, 2023 · 4 comments · Fixed by #32785
Labels
area:providers good first issue kind:feature Feature Requests provider:amazon-aws AWS/Amazon - related issues

Comments

@cw-somil
Copy link

Description

This feature adds support for Redshift Serverless clusters for the given operator.

Use case/motivation

RedshiftDataOperator currently only supports provisioned clusters since it has the capability of adding ClusterIdentifier as a parameter but not WorkgroupName. The addition of this feature would help users to use this operator for their serverless cluster as well.

Related issues

None

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@cw-somil cw-somil added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Jun 30, 2023
@boring-cyborg
Copy link

boring-cyborg bot commented Jun 30, 2023

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

@utkarsharma2 utkarsharma2 added provider:amazon-aws AWS/Amazon - related issues good first issue and removed needs-triage label for new issues that we didn't triage yet labels Jul 3, 2023
@vincbeck
Copy link
Contributor

vincbeck commented Jul 6, 2023

@shubham22

@shubham22
Copy link

Thanks for bringing this up @cw-somil. Valid concern and requires a minor update to the existing operator. We'll update here once the AWS team starts working on it. Meanwhile, others are welcome to work on it as well.

@ivica-k
Copy link
Contributor

ivica-k commented Jul 23, 2023

I'd like to take a stab at this one - Redshift Serverless offers $300 credits for the free trial so why not.

@shubham22 I may reach out to you via Slack just to get your team's opinion on the approach I took.

@ivica-k ivica-k mentioned this issue Jul 23, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:providers good first issue kind:feature Feature Requests provider:amazon-aws AWS/Amazon - related issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

adds support for querying Redshift Serverless clusters ivica-k/airflow
6 participants
@utkarsharma2 @ivica-k @cw-somil @eladkal @shubham22 @vincbeck