Skip to content

fix(bigframes): fix field name typos for ai.generate* functions - #17983

Open
sycai wants to merge 2 commits into
mainfrom
sycai_ai_generate_fix
Open

fix(bigframes): fix field name typos for ai.generate* functions#17983
sycai wants to merge 2 commits into
mainfrom
sycai_ai_generate_fix

Conversation

@sycai

@sycai sycai commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Fixes #17708 🦕

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request fixes a typo in several AI operations where "full_resposne" was corrected to "full_response", and adds corresponding integration tests to verify accessing the "full_response" field using the legacy Ibis compiler. The feedback suggests wrapping the yield statement in the use_ibis_compiler fixture within a try...finally block to ensure that the global SQL compiler setting is properly restored even if a test fails.

Comment thread packages/bigframes/tests/system/small/bigquery/test_ai.py Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@sycai
sycai marked this pull request as ready for review August 3, 2026 21:01
@sycai
sycai requested review from a team as code owners August 3, 2026 21:01
@sycai
sycai requested review from GarrettWu and tswast and removed request for a team August 3, 2026 21:01
@sycai
sycai requested a review from shuoweil August 4, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants