SameerMesiah97 opened a new pull request, #68597:
URL: https://github.com/apache/airflow/pull/68597

   **Description**
   
   This change refactors static connection parameter construction in 
SnowflakeHook by extracting configuration-specific logic from 
`_get_static_conn_params()` into dedicated helper methods.
   
   Connection parameter assembly has been decomposed into focused helpers 
responsible for base connection configuration, connector options, workload 
identity configuration, authentication configuration, and network 
configuration. `_get_static_conn_params()` remains the single entrypoint for 
constructing cached connection parameters while delegating individual 
configuration domains to dedicated helpers.
   
   **Rationale**
   
   `_get_static_conn_params()` had grown to encompass several distinct 
configuration responsibilities within a single method. Grouping related logic 
into dedicated helpers reduces method complexity, improves readability, and 
establishes clearer ownership boundaries for individual configuration paths.
   
   This structure makes future enhancements to authentication, workload 
identity, networking, and connector-specific configuration easier to implement 
and review independently.
   
   **Backwards Compatibility**
   
   This change is a refactor only and does not modify public APIs or runtime 
behavior. Connection parameter construction and authentication flows remain 
unchanged.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to