fix: nullpointer exception when executionerror is null (#754)

Praful Makani · web-flow · commit 03bda1bf7916 · 2020-09-24T11:34:41.000-04:00
diff --git a/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/Job.java b/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/Job.java
@@ -297,8 +297,10 @@ public TableResult getQueryResults(QueryResultsOption... options)
       job = reload();
     }
     if (job.getStatus() != null && job.getStatus().getError() != null) {
-      throw new JobException(
-          getJobId(), ImmutableList.copyOf(job.getStatus().getExecutionErrors()));
+      throw new BigQueryException(
+          job.getStatus().getExecutionErrors() == null
+              ? ImmutableList.of(job.getStatus().getError())
+              : ImmutableList.copyOf(job.getStatus().getExecutionErrors()));
     }
 
     // If there are no rows in the result, this may have been a DDL query.

Original file line number	Diff line number	Diff line change
`@@ -297,8 +297,10 @@ public TableResult getQueryResults(QueryResultsOption... options)`
`297`	`297`	`job = reload();`
`298`	`298`	`}`
`299`	`299`	`if (job.getStatus() != null && job.getStatus().getError() != null) {`
`300`		`- throw new JobException(`
`301`		`- getJobId(), ImmutableList.copyOf(job.getStatus().getExecutionErrors()));`
	`300`	`+ throw new BigQueryException(`
	`301`	`+ job.getStatus().getExecutionErrors() == null`
	`302`	`+ ? ImmutableList.of(job.getStatus().getError())`
	`303`	`+ : ImmutableList.copyOf(job.getStatus().getExecutionErrors()));`
`302`	`304`	`}`
`303`	`305`
`304`	`306`	`// If there are no rows in the result, this may have been a DDL query.`