Skip to content

fix(cli): backtick-quote BigQuery rows alias in list-tables query - #3337

Open
Tyagiquamar wants to merge 1 commit into
evidence-dev:mainfrom
Tyagiquamar:fix/bigquery-tables-rows-reserved-keyword
Open

Tyagiquamar wants to merge 1 commit into
evidence-dev:mainfrom
Tyagiquamar:fix/bigquery-tables-rows-reserved-keyword

Conversation

@Tyagiquamar

Copy link
Copy Markdown

Fixes #3331.

\evidence tables\ / \evidence schema\ always fail against a BigQuery direct connector: the introspection query aliases
ow_count AS rows, and
ows\ is a BigQuery reserved keyword, so the query never parses (\Syntax error: Unexpected keyword ROWS). Backtick-quote the alias, as the reporter confirmed by hand (\AS \
ows\\ returns rows correctly).

Validation (Docker node:22-bookworm, pnpm 9.15.0, ext4 clone):

  • New regression test in cli/cli/connection/index.test.ts fails on base source (1 failed / 17 passed) and passes with the fix (18/18).
  • tsc -p tsconfig.cli.json exit 0; eslint clean on both changed files; prettier core-options check clean (repo-config prettier run is blocked by the pre-existing generated file cli/src/routes/layout.css, which is absent from fresh clones and unrelated to this diff).

@vercel

vercel Bot commented Sep 18, 2026

Copy link
Copy Markdown

@Tyagiquamar is attempting to deploy a commit to the Evidence Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

evidence tables / evidence schema fail on BigQuery: rows is a reserved keyword

1 participant