-
Notifications
You must be signed in to change notification settings - Fork 427
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
feat: Add Dynamic Table Support for Stream Resource #2413
Comments
Hey @dcwangmit01. Thanks for creating the issue. We have existing resources' rework on our roadmap: https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/ROADMAP.md#our-roadmap. I will bump this resource's priority slightly. |
Hi. Is this something that I could help push forward by contributing code? My team needs this functionality in order to justify our use of the provider. Not having it means two tools instead of one for us. Thanks! |
Hey @josdotso. We are open to contributions! However, we do not have yet clear contribution guidelines and PR requirements written down inside the repository. They are still to come in the upcoming weeks. If this is not a problem for you, we will be happy to review the change. |
Hey, What is the update on that feature ? |
Hey @AZenat. No update from our side yet. However, we have described the contribution guidelines: https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/CONTRIBUTING.md#contributing, so it should be easier to make a contribution. :) |
Thanks @sfc-gh-asawicki, I will try to do it then. |
Hi @AZenat 👋 |
Terraform CLI and Provider Versions
Use Cases or Problem Statement
Currently the Streams resource may be created
on_table
,on_view
, andon_stage
. However, there is no support foron_dynamic_table
.Effectively, the request is to enable the following in Terraform.
Relevant Issue:
#1922
Proposal
Enable this capability for creating streams on Dynamic Tables in the same way that
on_table
,on_view
, andon_stage
are supported.How much impact is this issue causing?
Medium
Additional Information
No response
The text was updated successfully, but these errors were encountered: