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

Refactor Azure Synapse Pipeline Hook and Add Managed Identity Support #45894

Closed
1 of 2 tasks
jjavieralonso opened this issue Jan 22, 2025 · 1 comment
Closed
1 of 2 tasks
Labels
AI Spam area:providers kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet provider:microsoft-azure Azure-related issues

Comments

@jjavieralonso
Copy link

Description

This feature introduces a new base class BaseAzureSynapseHook to manage common connection forms and functionalities for Azure Synapse Hooks. It also adds managed identity support for DefaultAzureCredential and deprecates the old default connection name azure_synapse_connection.

Use case/motivation

The refactoring improves code maintainability by centralizing common logic in a base class, reducing duplication, and making it easier to manage changes. The addition of managed identity support enhances security by allowing Azure Synapse connections to use managed identities instead of client secrets.

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@jjavieralonso jjavieralonso added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Jan 22, 2025
@potiuk
Copy link
Member

potiuk commented Jan 22, 2025

This looks like AI-generated. useless issue report that brings no value and makes no sense. We are generally blocking users that sends a lot of spam AI reports generated by bots.. as of yesterday so we will report your account and block it unless:

a) you explain how you generated reports
b) prove you are human
c) explain why you created the issue

@potiuk potiuk closed this as completed Jan 22, 2025
@potiuk potiuk added the AI Spam label Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI Spam area:providers kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet provider:microsoft-azure Azure-related issues
Projects
None yet
Development

No branches or pull requests

2 participants