-
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
Better Azure integration - Pipe + Notification Integration #553
Better Azure integration - Pipe + Notification Integration #553
Conversation
Can you run |
…ovider-snowflake into better_azure_integration
Sorry I missed that. Docs updated |
/ok-to-test sha=5e07f99 |
Integration tests failure for 5e07f99 |
Tests should be fixed after last commit |
/ok-to-test sha=bd59143 |
Integration tests failure for bd59143 |
Oops my bad. Can you resolve conflicts and I'll retetst? I just merged another PR. |
…ovider-snowflake into better_azure_integration
Sure. Resolved |
/ok-to-test sha=716c4f8 |
Integration tests success for 716c4f8 |
We needed better Azure integration, there are two useful changes:
Test Plan
Notification integration was only tested with Azure. It wasn't tested with GCP
References
Notification Integration - https://docs.snowflake.com/en/sql-reference/sql/create-notification-integration.html
Pipe - https://docs.snowflake.com/en/sql-reference/sql/create-pipe.html
*