tree 82be3fee9d8c7704ae3087e1d6009f3953176618
parent a81ec08a81f824c73922bab715a32f19afb8f80e
author shollyman <shollyman@google.com> 1601487234 -0700
committer GitHub <noreply@github.com> 1601487234 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJfdMGCCRBK7hj4Ov3rIwAAdHIIAEwaxcHBZBHRzQPENwt9BnqY
 IC6odulCSUniWW33YgGxgZQ6Hscu0EVjOhtBC7yviFpvsIHbNluvOnHtGOU1oiUz
 8YERSlP+wzNh9sEt9Kx9MUZQnGPzVa4r/Ylchfdo89e6PxPAOz72VUWPDY4DSM+q
 m2/kWx1Ed/GDrsRDj+kZhmppAuLMUwAO9+LHjLLnbX5m+7kxfN2+QD/wdStMWCYT
 F2eW70qqP8S0Zmkj3aaqwcIZX4Dxnu/17z4pxIrKvz9x2wkOMcRjF5gJgJrNPh6u
 s9IBImLjPYLedDl165nWO+Mv9A0+kVaWX2N4FPgfaMCVVv5GNEO9ET1BNcxR0TA=
 =tJOI
 -----END PGP SIGNATURE-----
 

fix(bigquery): address issue with directly Read()'ing from a Query. (#2940)

Existing query test cases in integration tests weren't running long
enough to validate we were successful in falling back to the classic
query path.

PR adds a cheap-but-slow test query to exercise slower queries, and
fixed the issue reported in #2937 where we weren't passing enough of
a job object to the classic path to allow probing successfully.

Alternatives considered:
* Add a jobs.get call under the hood to get the job config (adds
  unnecessary latency)
* Remove isQueryJob() test (can consider this in the future, but
  expands the coverage area).

Fixes: #2937

Co-authored-by: Alex Hong <9397363+hongalex@users.noreply.github.com>