Don't set location attribute on CopyJobConfig

evansd · evansd · commit f329a4a7f0c3 · 2022-12-02T14:17:00.000Z
As far as I can tell, this has never been a valid config value here. But prior to the merging of [this PR][1] in v2.7.0, invalid attributes were silently ignored. [1]: googleapis/python-bigquery#446
diff --git a/openprescribing/gcutils/bigquery.py b/openprescribing/gcutils/bigquery.py
@@ -382,13 +382,11 @@ def delete_all_rows(self, **options):
         self.run_job("query", args, options, default_options)
 
     def copy_to_new_dataset(self, new_dataset_key, **options):
-        default_options = {"location": settings.BQ_LOCATION}
-
         dataset_ref = self.gcbq_client.dataset(DATASETS[new_dataset_key])
         new_table_ref = dataset_ref.table(self.table_id)
 
         args = [self.gcbq_table_ref, new_table_ref]
-        self.run_job("copy_table", args, options, default_options)
+        self.run_job("copy_table", args, options, default_options={})
 
     def move_to_new_dataset(self, new_dataset_id):
         self.copy_to_new_dataset(new_dataset_id)
