Merge pull request #644 from aozarov/master

mziccard · mziccard · commit 47dcc63eb9b7 · 2016-02-12T09:34:36.000+01:00
Revert #625 and Fixes #624
diff --git a/gcloud-java-bigquery/src/test/java/com/google/gcloud/bigquery/it/ITBigQueryTest.java b/gcloud-java-bigquery/src/test/java/com/google/gcloud/bigquery/it/ITBigQueryTest.java
@@ -50,6 +50,7 @@
 import com.google.gcloud.bigquery.InsertAllResponse;
 import com.google.gcloud.bigquery.Job;
 import com.google.gcloud.bigquery.JobInfo;
+import com.google.gcloud.bigquery.JobStatistics;
 import com.google.gcloud.bigquery.LoadJobConfiguration;
 import com.google.gcloud.bigquery.QueryJobConfiguration;
 import com.google.gcloud.bigquery.QueryRequest;
@@ -683,10 +684,9 @@ public void testQuery() throws InterruptedException {
       rowCount++;
     }
     assertEquals(2, rowCount);
-    // todo(mziccard) uncomment as soon as #624 is closed
-    // Job queryJob = bigquery.getJob(response.jobId());
-    // JobStatistics.QueryStatistics statistics = queryJob.statistics();
-    // assertNotNull(statistics.queryPlan());
+    Job queryJob = bigquery.getJob(response.jobId());
+    JobStatistics.QueryStatistics statistics = queryJob.statistics();
+    assertNotNull(statistics.queryPlan());
   }
 
   @Test
@@ -851,10 +851,9 @@ public void testQueryJob() throws InterruptedException {
     }
     assertEquals(2, rowCount);
     assertTrue(bigquery.delete(DATASET, tableName));
-    // todo(mziccard) uncomment as soon as #624 is closed
-    // Job queryJob = bigquery.getJob(remoteJob.jobId());
-    // JobStatistics.QueryStatistics statistics = queryJob.statistics();
-    // assertNotNull(statistics.queryPlan());
+    Job queryJob = bigquery.getJob(remoteJob.jobId());
+    JobStatistics.QueryStatistics statistics = queryJob.statistics();
+    assertNotNull(statistics.queryPlan());
   }
 
   @Test
