docs(config): document the BigQuery server setup - #340
Merged
Conversation
Lead the BigQuery section with the shape a deployment actually uses — a
projectId plus a serviceAccountKeyJson reference to an environment
variable — and document that the same file needs no local variant: an
unset {"env": ...} reference is omitted, so the connection falls back to
application default credentials.
Also warn against the near-miss that fallback invites: base64-ing
~/.config/gcloud/application_default_credentials.json into the variable.
That file is an authorized_user credential, and the connector rejects it
for having no client_email and private_key.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01APyHsXjok5JD79hH4sjvbH
Signed-off-by: Lloyd Tabb <lloyd.tabb@gmail.com>
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01APyHsXjok5JD79hH4sjvbH Signed-off-by: Lloyd Tabb <lloyd.tabb@gmail.com>
lloydtabb
force-pushed
the
claude/bigquery-docs-update-6vq1ng
branch
from
August 20, 2026 17:34
5434e22 to
c25a053
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replaces the BigQuery prose in
setup/config.malloynbwith the four steps a server deployment actually takes:malloy-config.jsonat an environment variable for the key.gcloud auth login --update-adcin a test environment.setup/extension.malloynbgets a one-line pointer from its BigQuery section.Note on step 2
As written, step 2 does not work against the connector today.
credentialsFromJsoninbigquery_connection.tsaccepts only a service account key (client_email+private_key) or anexternal_accountconfig, andapplication_default_credentials.jsonis anauthorized_usercredential, so it is rejected withserviceAccountKeyJson parsed but is not a service account key.Documented this way deliberately — the connector is the piece that needs to widen.
Verification
npm run buildcompiles clean. The newconfig.malloynb#bigquery-google-bigqueryanchor and the link to it fromextension.malloynbboth resolve in the generated HTML.Generated by Claude Code