From d67387fefc99de60fc2875b914ddf30ee34005e4 Mon Sep 17 00:00:00 2001
From: PhongChuong <phong.hoa.chuong@gmail.com>
Date: Mon, 22 Apr 2024 13:52:16 -0400
Subject: [PATCH 1/3] fix: Remove @InternalApi from TableResult

Since users depends on TableResult in testing, removing @InternalApi forces maintainers to change/roll out changes to TableResult more carefully in the future.
---
 .../src/main/java/com/google/cloud/bigquery/TableResult.java     | 1 -
 1 file changed, 1 deletion(-)

diff --git a/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/TableResult.java b/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/TableResult.java
index 203a91fab..7a5591909 100644
--- a/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/TableResult.java
+++ b/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/TableResult.java
@@ -27,7 +27,6 @@
 import java.util.Objects;
 import javax.annotation.Nullable;
 
-@InternalApi
 @AutoValue
 public abstract class TableResult implements Page<FieldValueList>, Serializable {
 

From be7528d21c3d14806058619dbad97d1bb70b1488 Mon Sep 17 00:00:00 2001
From: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Date: Mon, 22 Apr 2024 17:54:57 +0000
Subject: [PATCH 2/3] =?UTF-8?q?=F0=9F=A6=89=20Updates=20from=20OwlBot=20po?=
 =?UTF-8?q?st-processor?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
---
 .../src/main/java/com/google/cloud/bigquery/TableResult.java     | 1 -
 1 file changed, 1 deletion(-)

diff --git a/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/TableResult.java b/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/TableResult.java
index 7a5591909..1b4b0e101 100644
--- a/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/TableResult.java
+++ b/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/TableResult.java
@@ -16,7 +16,6 @@
 
 package com.google.cloud.bigquery;
 
-import com.google.api.core.InternalApi;
 import com.google.api.gax.paging.Page;
 import com.google.auto.value.AutoValue;
 import com.google.common.base.Function;

From 49972b1f24282304ac24731eb544f385f40f6668 Mon Sep 17 00:00:00 2001
From: PhongChuong <phong.hoa.chuong@gmail.com>
Date: Mon, 22 Apr 2024 14:05:07 -0400
Subject: [PATCH 3/3] Remove clirr ignore difference entries for TableResult

---
 .../clirr-ignored-differences.xml             | 38 -------------------
 1 file changed, 38 deletions(-)

diff --git a/google-cloud-bigquery/clirr-ignored-differences.xml b/google-cloud-bigquery/clirr-ignored-differences.xml
index abf827b48..9c69fd6a6 100644
--- a/google-cloud-bigquery/clirr-ignored-differences.xml
+++ b/google-cloud-bigquery/clirr-ignored-differences.xml
@@ -2,17 +2,6 @@
 <!-- see http://www.mojohaus.org/clirr-maven-plugin/examples/ignored-differences.html -->
 <differences>
   <!-- TODO: REMOVE AFTER RELEASE -->
-  <difference>
-    <differenceType>3005</differenceType>
-    <className>com/google/cloud/bigquery/TableResult*</className>
-    <justification>TableResult is an internal API and it should be fine to update</justification>
-  </difference>
-  <difference>
-    <differenceType>7002</differenceType>
-    <className>com/google/cloud/bigquery/TableResult*</className>
-    <method>*TableResult(*)</method>
-    <justification>TableResult is an internal API and it should be fine to update</justification>
-  </difference>
   <difference>
     <differenceType>7004</differenceType>
     <className>com/google/cloud/bigquery/spi/v2/BigQueryRpc</className>
@@ -25,19 +14,6 @@
     <method>com.google.api.services.bigquery.model.GetQueryResultsResponse getQueryResultsWithRowLimit(java.lang.String, java.lang.String, java.lang.String, java.lang.Integer)</method>
     <justification>getQueryResultsWithRowLimit is just used by ConnectionImpl at the moment so it should be fine to update the signature instead of writing an overloaded method</justification>
   </difference>
-  <difference>
-    <differenceType>7004</differenceType>
-    <className>com/google/cloud/bigquery/TableResult*</className>
-    <method>*TableResult(*)</method>
-    <justification>It should be fine to update TableResult constructors since it is used to return results to the user and users should not directly construct TableResult objects</justification>
-  </difference>
-  <difference>
-    <differenceType>7005</differenceType>
-    <className>com/google/cloud/bigquery/TableResult*</className>
-    <method>*TableResult(*)</method>
-    <to>*TableResult(*)</to>
-    <justification>It should be fine to update TableResult constructors since it is used to return results to the user and users should not directly construct TableResult objects</justification>
-  </difference>
   <difference>
     <differenceType>7013</differenceType>
     <className>com/google/cloud/bigquery/ExternalTableDefinition*</className>
@@ -58,16 +34,6 @@
     <className>com/google/cloud/bigquery/TableInfo*</className>
     <method>*ResourceTags(*)</method>
   </difference>
-  <difference>
-    <differenceType>7013</differenceType>
-    <className>com/google/cloud/bigquery/TableResult*</className>
-    <method>*getPageNoSchema(*)</method>
-  </difference>
-  <difference>
-    <differenceType>7013</differenceType>
-    <className>com/google/cloud/bigquery/TableResult*</className>
-    <method>*toBuilder(*)</method>
-  </difference>
   <difference>
     <differenceType>7012</differenceType>
     <className>com/google/cloud/bigquery/Connection</className>
@@ -163,8 +129,4 @@
     <className>com/google/cloud/bigquery/StandardTableDefinition*</className>
     <method>*BigLakeConfiguration(*)</method>
   </difference>
-  <difference>
-    <differenceType>8001</differenceType>
-    <className>com/google/cloud/bigquery/EmptyTableResult*</className>
-  </difference>
   </differences>
\ No newline at end of file