The GitHub Actions job "Tests" on airflow.git/main has failed.
Run started by GitHub user potiuk (triggered by potiuk).

Head commit for run:
3ab2eb9884c74696051e7617b9ebbd46315da7b5 / Shivam Rastogi 
<[email protected]>
fix(amazon): Filter kwargs in AthenaSQLHook to prevent TypeError (#62227)

* fix(amazon): Filter kwargs in AthenaSQLHook to prevent TypeError

BaseSQLOperator.get_hook() passes all connection extra_dejson fields as
kwargs to the hook constructor. Athena-specific params like s3_staging_dir
and work_group are not accepted by AwsGenericHook.__init__(), causing a
TypeError. Use an allowlist to only forward valid AWS-generic params to
the parent class.

Closes: #55678

* fix: run prek

* fix(athena): Declare explicit constructor params in AthenaSQLHook

Replace kwargs filtering approach with explicit named parameters in
AthenaSQLHook.__init__ to match AwsGenericHook signature. This fixes
the TypeError from #55678 where BaseSQLOperator.get_hook() passed
connection extras like s3_staging_dir as kwargs.

The constructor now declares aws_conn_id, verify, region_name,
client_type, resource_type, and config as named params with types
matching the parent. Unexpected kwargs from connection extras are
absorbed by **kwargs and discarded.

Update ALLOWED_THICK_HOOKS_PARAMETERS in test_hooks_signature.py
to permit the newly declared params. Add config forwarding assertion
to existing test.

Report URL: https://github.com/apache/airflow/actions/runs/22436498286

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to