mitre88 opened a new pull request, #70486: URL: https://github.com/apache/airflow/pull/70486
`aws_conn_id` and `redshift_data_api_kwargs` are template fields of `S3ToRedshiftOperator`, but `__init__` applied logic to both before Jinja rendering: - the NOTSET/default resolution captured the un-rendered `aws_conn_id` into `self._aws_conn_id`, so a templated connection id was used verbatim at execute time; - the forbidden-keys check (`sql`, `parameters`) never saw a templated kwargs dict. Both now run at the top of `execute()` as locals (`conn_set` / `aws_conn_id` were only read from `execute()`), and the existing kwargs test asserts the failure at execute time. Removes the class from the exemption list. related: #70296 --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Fable 5) Generated-by: Claude Code (Fable 5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
