Merge 0.36.8.2 -> release-x.37.x (#13599)

* Specify location on getQueryResults calls (#13592)

You must specify the location on any getQueryResults calls to BigQuery.
It's supposed to be optional for anything but EU and US regions, but
useast1 seems to require it.

Fixes #13582

[ci bigquery]

* v0.36.8.2

Co-authored-by: Robert Roland <rob@metabase.com>

bin/version

 #!/usr/bin/env bash
 
-VERSION='v1.37.0'
+VERSION='v0.37.0'
 
 # dynamically pull more interesting stuff from latest git commit
 HASH=$(git show-ref --head --hash=7 head)            # first 7 letters of hash should be enough; that's what GitHub uses

modules/drivers/bigquery/src/metabase/driver/bigquery.clj

@@ -195,12 +195,13 @@
       ~@body)))
 
 (defn- ^GetQueryResultsResponse get-query-results
-  [^Bigquery client ^String project-id ^String job-id ^String page-token]
+  [^Bigquery client ^String project-id ^String job-id ^String location ^String page-token]
   (when page-callback
     (page-callback))
   (let [request (doto (.getQueryResults (.jobs client) project-id job-id)
                   (.setMaxResults max-results-per-page)
-                  (.setPageToken page-token))]
+                  (.setPageToken page-token)
+                  (.setLocation location))]
     (google/execute request)))
 
 (defn- ^GetQueryResultsResponse execute-bigquery
@@ -217,10 +218,11 @@
                    (bigquery.params/set-parameters! parameters))
          query-response ^QueryResponse (google/execute (.query (.jobs client) project-id request))
          job-ref (.getJobReference query-response)
+         location (.getLocation job-ref)
          job-id (.getJobId job-ref)
          proj-id (.getProjectId job-ref)]
      (with-finished-response [_ query-response]
-       (get-query-results client proj-id job-id nil)))))
+       (get-query-results client proj-id job-id location nil)))))
 
 (defn- post-process-native
   "Parse results of a BigQuery query. `respond` is the same function passed to
@@ -254,6 +256,7 @@
                     (with-finished-response [next-resp (get-query-results (database->client database)
                                                                           (.getProjectId (.getJobReference response))
                                                                           (.getJobId (.getJobReference response))
+                                                                          (.getLocation (.getJobReference response))
                                                                           next-page-token)]
                       (fetch-page next-resp)))))]
          (for [^TableRow row (fetch-page response)]

modules/drivers/bigquery/test/metabase/test/data/bigquery.clj

@@ -152,7 +152,8 @@
         job-ref (.getJobReference query-response)
         job-id (.getJobId job-ref)
         proj-id (.getProjectId job-ref)
-        response (#'bigquery/get-query-results client proj-id job-id nil)]
+        location (.getLocation job-ref)
+        response (#'bigquery/get-query-results client proj-id job-id location nil)]
     (#'bigquery/post-process-native @details respond response)))
 
 (defprotocol ^:private Insertable