shahar1 opened a new pull request, #66962: URL: https://github.com/apache/airflow/pull/66962
The `BigQueryStreamingBufferEmptySensor` added in #66652 passed the legacy `project:dataset.table` string (colon separator) to `Client.get_table`. The BigQuery client only accepts a `TableReference` or a standard-SQL `project.dataset.table` string, so against real BigQuery `poke()` always raised `NotFound` and the `example_bigquery_sensors` system test could never get past `check_streaming_buffer_empty`. This builds an explicit `TableReference` instead, mirroring how `BigQueryHook.insert_all` constructs its table reference. The legacy colon form is kept only for log/error messages, where it is the conventional BigQuery display format. The unit test previously asserted the colon string was passed to `get_table`, which masked the bug since the mock accepted anything — it now asserts a proper `TableReference` is used. related: #66652 --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Opus 4.7) Generated-by: Claude Code (Opus 4.7) 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]
