SEPURI-SAI-KRISHNA opened a new pull request, #71648:
URL: https://github.com/apache/airflow/pull/71648
`BigQueryTableExistenceSensor` and `BigQueryTablePartitionExistenceSensor`
hand their
impersonation chain to the trigger inside `hook_params`:
```python
trigger=BigQueryTableExistenceTrigger(
...,
hook_params={"impersonation_chain": self.impersonation_chain},
)
```
but the trigger authenticates from a different attribute entirely:
```python
def _get_async_hook(self) -> BigQueryTableAsyncHook:
return BigQueryTableAsyncHook(
gcp_conn_id=self.gcp_conn_id,
impersonation_chain=self.impersonation_chain
)
```
Nothing ever sets `self.impersonation_chain` from those call sites, so it
stays `None` and
the deferred existence check runs as the connection's service account.
`hook_params` is
stored and serialized but never read anywhere in the module — it carries the
value and
then drops it.
`poke()` builds its hook correctly, so the sensor honours impersonation when
`deferrable=False` and ignores it when `deferrable=True`. That divergence is
the awkward
part: turning deferrable off appears to "fix" the permissions error, which
points
investigation away from the real cause.
The trigger already grew a proper `impersonation_chain` parameter in #36341;
the sensors
were simply never moved onto it. This change passes it explicitly at both
defer sites.
`hook_params` is left untouched here. It is a required argument on the
trigger's public
`__init__`, and now that nothing reads it, removing or deprecating it is a
separate
decision — happy to follow up if maintainers would like it gone.
For reviewers: `GoogleBaseHook.__init__` falls back to the connection's
`impersonation_chain` extra when the argument is absent, so deployments that
configure
impersonation on the connection were never affected. This only bites when
impersonation is
set on the sensor, which is the documented parameter.
No newsfragment: this is a provider change, and provider changelogs are
regenerated from
`git log` by the release manager.
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Claude Code (Opus 5)
Generated-by: Claude Code (Opus 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]
