ambika-garg commented on code in PR #36622: URL: https://github.com/apache/airflow/pull/36622#discussion_r1444122270
########## airflow/providers/microsoft/azure/hooks/synapse.py: ########## @@ -240,53 +296,22 @@ class AzureSynapsePipelineRunException(AirflowException): """An exception that indicates a pipeline run failed to complete.""" -class AzureSynapsePipelineHook(BaseHook): +class AzureSynapsePipelineHook(BaseAzureSynapseHook): """ A hook to interact with Azure Synapse Pipeline. :param azure_synapse_conn_id: The :ref:`Azure Synapse connection id<howto/connection:synapse>`. :param azure_synapse_workspace_dev_endpoint: The Azure Synapse Workspace development endpoint. """ - conn_type: str = "azure_synapse_pipeline" - conn_name_attr: str = "azure_synapse_conn_id" - default_conn_name: str = "azure_synapse_connection" - hook_name: str = "Azure Synapse Pipeline" - - @classmethod - def get_connection_form_widgets(cls) -> dict[str, Any]: - """Returns connection widgets to add to connection form.""" - from flask_appbuilder.fieldwidgets import BS3TextFieldWidget - from flask_babel import lazy_gettext - from wtforms import StringField - - return { - "tenantId": StringField(lazy_gettext("Tenant ID"), widget=BS3TextFieldWidget()), - } - - @classmethod - def get_ui_field_behaviour(cls) -> dict[str, Any]: - """Returns custom field behaviour.""" - return { - "hidden_fields": ["schema", "port", "extra"], - "relabeling": {"login": "Client ID", "password": "Secret", "host": "Synapse Workspace URL"}, - } + default_conn_name: str = "azure_synapse_default" Review Comment: Ah right, I understand it would be a breaking change. So, what would be the best practice to change the `default_conn_name`? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org