---

mziccard · mziccard · commit da09b1a31e25 · 2015-11-19T19:11:07.000+01:00
yaml --- r: 6423 b: refs/heads/tswast-patch-1 c: 12aa555 h: refs/heads/master i: 6421: f13cfe3 6419: 3189519 6415: a74ee99
diff --git a/[refs] b/[refs]
@@ -57,5 +57,5 @@ refs/tags/v0.18.0: 9d193c4c4b9d1c6f21515dd8e50836b9194ec9bb
 refs/tags/v0.19.0: e67b56e4d8dad5f9a7b38c9b2107c23c828f2ed5
 refs/tags/v0.20.0: 839f7fb7156535146aa1cb2c5aadd8d375d854e8
 refs/tags/v0.20.1: 370471f437f1f4f68a11e068df5cd6bf39edb1fa
-refs/heads/tswast-patch-1: e3a99c2f9314f938bafd9fd5d19a98a43335bd98
+refs/heads/tswast-patch-1: 12aa55591cde5f0f5b3bbf6e4d29c7d049925c5f
 refs/heads/pubsub-streaming-pull: 19262b752ee874eb2ca3b950eb2aef44d5a5267b
diff --git a/branches/tswast-patch-1/gcloud-java-bigquery/src/main/java/com/google/gcloud/spi/DefaultBigQueryRpc.java b/branches/tswast-patch-1/gcloud-java-bigquery/src/main/java/com/google/gcloud/spi/DefaultBigQueryRpc.java
@@ -35,6 +35,7 @@
 import com.google.api.services.bigquery.model.Job;
 import com.google.api.services.bigquery.model.JobList;
 import com.google.api.services.bigquery.model.JobReference;
+import com.google.api.services.bigquery.model.JobStatus;
 import com.google.api.services.bigquery.model.QueryRequest;
 import com.google.api.services.bigquery.model.QueryResponse;
 import com.google.api.services.bigquery.model.Table;
@@ -220,8 +221,9 @@ public Tuple<String, Iterable<Table>> listTables(String datasetId, Map<Option, ?
           .setMaxResults(MAX_RESULTS.getLong(options))
           .setPageToken(PAGE_TOKEN.getString(options))
           .execute();
+      Iterable<TableList.Tables> tables = tableList.getTables();
       return Tuple.of(tableList.getNextPageToken(),
-          Iterables.transform(tableList.getTables(),
+          Iterables.transform(tables != null ? tables : ImmutableList.<TableList.Tables>of(),
               new Function<TableList.Tables, Table>() {
                 @Override
                 public Table apply(TableList.Tables f) {
@@ -350,11 +352,19 @@ public Tuple<String, Iterable<Job>> listJobs(Map<Option, ?> options) throws BigQ
           .setPageToken(PAGE_TOKEN.getString(options))
           .setProjection(DEFAULT_PROJECTION)
           .execute();
+      Iterable<JobList.Jobs> jobs = jobsList.getJobs();
       return Tuple.of(jobsList.getNextPageToken(),
-          Iterables.transform(jobsList.getJobs(),
+          Iterables.transform(jobs != null ? jobs : ImmutableList.<JobList.Jobs>of(),
               new Function<JobList.Jobs, Job>() {
                 @Override
                 public Job apply(JobList.Jobs f) {
+                  JobStatus statusPb = f.getStatus() != null ? f.getStatus() : new JobStatus();
+                  if (statusPb.getState() == null) {
+                    statusPb.setState(f.getState());
+                  }
+                  if (statusPb.getErrorResult() == null) {
+                    statusPb.setErrorResult(f.getErrorResult());
+                  }
                   return new Job()
                       .setConfiguration(f.getConfiguration())
                       .setId(f.getId())
