New Resource: azurerm_data_factory_dataset_snowflake
#11116
Conversation
ghost
added
orestiskats19
changed the title
azurerm_data_factory_dataset_snowflake #11051azurerm_data_factory_dataset_snowflake
orestiskats19
force-pushed
the
azurerm_data_factory_dataset_snowflake
branch
from
52ce342 to
d09e732
Compare
…wflake' into azurerm_data_factory_dataset_snowflake # Conflicts: # website/docs/r/data_factory_dataset_snowflake.html.markdown
There was a problem hiding this comment.
Hey @orestiskats19! This PR is looking really good. I have a couple of asks from you and then we can merge it in!
azurerm/internal/services/datafactory/data_factory_dataset_snowflake_resource.go
Outdated
Show resolved
Hide resolved
azurerm/internal/services/datafactory/data_factory_dataset_snowflake_resource.go
Outdated
Show resolved
Hide resolved
azurerm/internal/services/datafactory/data_factory_dataset_snowflake_resource.go
Outdated
Show resolved
Hide resolved
azurerm/internal/services/datafactory/data_factory_dataset_snowflake_resource.go
Outdated
Show resolved
Hide resolved
azurerm/internal/services/datafactory/data_factory_dataset_snowflake_resource_test.go
Outdated
Show resolved
Hide resolved
azurerm/internal/services/datafactory/data_factory_dataset_snowflake_resource_test.go
Outdated
Show resolved
Hide resolved
|
Hey @mbfrahry, thanks for reviewing my PR. I have made the changes that you suggested, let me know what you think 😃 . |
|
Thanks for addressing the review @orestiskats19! It should go out in 2.57.0 of the provider |
|
This has been released in version 2.57.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example: provider "azurerm" {
version = "~> 2.57.0"
}
# ... other configuration ... |
|
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
New Resource for Snowflake table in data factory service:
azurerm_data_factory_dataset_snowflake.Resolves issue #11051 .