Cache job constructed for a synchronous query.#1675
Cache job constructed for a synchronous query.#1675tseaver merged 1 commit intogoogleapis:masterfrom tseaver:1673-bigquery-reify_query_job
Conversation
|
Code itself LGTM but before merging here is a design question: should the user have a way to bust our cached value? |
I wouldn't think so. The |
|
But what if the |
That is what the
No different than holding onto any other client-side proxy for a server-side resource. The docs for
which implies that the job must last that long, too (since its ID is used to fetch the results). If the user tries to fetch results after the job / result table is gone, then the API will (presumably) raise a 404. |
|
LGTM |
Fixes #1673.