Skip to content

Remove redundant Snowflake prefixes from Cortex Agent and SQL API modules - #74300

Open
SameerMesiah97 wants to merge 1 commit into
apache:mainfrom
SameerMesiah97:Remove-Snowflake-Prefix
Open

SameerMesiah97 wants to merge 1 commit into
apache:mainfrom
SameerMesiah97:Remove-Snowflake-Prefix

Conversation

@SameerMesiah97

@SameerMesiah97 SameerMesiah97 commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Description

This change renames the Snowflake Cortex Agent hook and operator modules to cortex_agent.py, and the SQL API hook module to sql_api.py.

Imports, mock targets, corresponding test filenames, documentation and provider metadata are updated to use the new module paths.

Rationale

In line with AIP-21, this change removes redundant provider prefixes from module names. The snowflake_ prefix repeats information already provided by the airflow.providers.snowflake package.

Removing it makes these module paths shorter while retaining descriptive filenames and keeping class names unchanged.

Tests

  • Updated existing hook and operator tests to use the new module paths.
  • Added compatibility tests verifying that old imports emit deprecation warnings and resolve to the same classes as the new imports.

Backwards Compatibility

Existing import paths remain available through add_deprecated_classes() mappings in the hook and operator packages. Existing DAGs continue to work, with deprecation warnings directing users to the new paths.

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Generated-by: [GPT 5.6] following the guidelines

@boring-cyborg boring-cyborg Bot added area:providers kind:documentation provider:snowflake Issues related to Snowflake provider labels Oct 5, 2026
@SameerMesiah97
SameerMesiah97 marked this pull request as ready for review October 5, 2026 21:32
@SameerMesiah97
SameerMesiah97 requested a review from potiuk as a code owner October 5, 2026 21:32
@SameerMesiah97
SameerMesiah97 marked this pull request as draft October 5, 2026 21:32
@SameerMesiah97
SameerMesiah97 force-pushed the Remove-Snowflake-Prefix branch from 443dc37 to e7ec202 Compare October 6, 2026 18:15
@SameerMesiah97
SameerMesiah97 marked this pull request as ready for review October 6, 2026 18:15
@SameerMesiah97
SameerMesiah97 force-pushed the Remove-Snowflake-Prefix branch from e7ec202 to 6f22e0c Compare October 6, 2026 20:55
@SameerMesiah97
SameerMesiah97 marked this pull request as draft October 7, 2026 17:25
@SameerMesiah97
SameerMesiah97 force-pushed the Remove-Snowflake-Prefix branch 2 times, most recently from 059587a to ecef787 Compare October 9, 2026 17:32
@SameerMesiah97
SameerMesiah97 marked this pull request as ready for review October 9, 2026 17:43
@SameerMesiah97
SameerMesiah97 marked this pull request as draft October 10, 2026 14:48
@SameerMesiah97
SameerMesiah97 force-pushed the Remove-Snowflake-Prefix branch from ecef787 to 9fb0625 Compare October 10, 2026 16:24
Rename Cortex Agent and SQL API modules and update their references.
Preserve existing import paths through deprecated aliases.
@SameerMesiah97
SameerMesiah97 force-pushed the Remove-Snowflake-Prefix branch from 9fb0625 to d9cef88 Compare October 10, 2026 17:05
@SameerMesiah97
SameerMesiah97 marked this pull request as ready for review October 10, 2026 20:20

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant