Merge pull request #211 from jswarburton/handle-flo-bq-errors

narape · web-flow · commit 98d3be496b8c · 2019-12-04T15:24:18.000+01:00
Reload BigQuery job after it is done in order to correctly capture errors
diff --git a/contrib/flo-bigquery/src/main/java/com/spotify/flo/contrib/bigquery/DefaultBigQueryClient.java b/contrib/flo-bigquery/src/main/java/com/spotify/flo/contrib/bigquery/DefaultBigQueryClient.java
@@ -78,6 +78,7 @@ public JobInfo job(JobInfo jobInfo, JobOption... options) {
       }
       job = job.reload();
     }
+    job = job.reload();
 
     final BigQueryError error = job.getStatus().getError();
     if (error != null) {

Original file line number	Diff line number	Diff line change
`@@ -78,6 +78,7 @@ public JobInfo job(JobInfo jobInfo, JobOption... options) {`
`78`	`78`	`}`
`79`	`79`	`job = job.reload();`
`80`	`80`	`}`
	`81`	`+ job = job.reload();`
`81`	`82`
`82`	`83`	`final BigQueryError error = job.getStatus().getError();`
`83`	`84`	`if (error != null) {`