feat: add fields and the BackupType proto for Hot Backups (#2300)

* chore: add grpc-java dependencies to WORKSPACE and update gapic-generator-java to 2.43.0 PiperOrigin-RevId: 656487430 Source-Link: googleapis/googleapis@cf16946 Source-Link: https://github.com/googleapis/googleapis-gen/commit/4b4c8ab484b34251b142ad17d14e25a33f32c1b0 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNGI0YzhhYjQ4NGIzNDI1MWIxNDJhZDE3ZDE0ZTI1YTMzZjMyYzFiMCJ9 * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * feat: add fields and the BackupType proto for Hot Backups docs: clarify comments and fix typos PiperOrigin-RevId: 658791576 Source-Link: googleapis/googleapis@c93b54f Source-Link: https://github.com/googleapis/googleapis-gen/commit/e52ba38a95a82f7588d0dd3a2284c98850dab9e1 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZTUyYmEzOGE5NWE4MmY3NTg4ZDBkZDNhMjI4NGM5ODg1MGRhYjllMSJ9 * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
googleapis · Aug 12, 2024 · acaa3ff · acaa3ff
1 parent 715b7d0
commit acaa3ff
Show file tree

Hide file tree

Showing 16 changed files with 1,011 additions and 145 deletions.
diff --git a/...gtable/src/main/java/com/google/cloud/bigtable/admin/v2/BaseBigtableTableAdminClient.java b/...gtable/src/main/java/com/google/cloud/bigtable/admin/v2/BaseBigtableTableAdminClient.java
@@ -4068,7 +4068,7 @@ public final UnaryCallable<ListBackupsRequest, ListBackupsResponse> listBackupsC
    * Create a new table by restoring from a completed backup. The returned table [long-running
    * operation][google.longrunning.Operation] can be used to track the progress of the operation,
    * and to cancel it. The [metadata][google.longrunning.Operation.metadata] field type is
-   * [RestoreTableMetadata][google.bigtable.admin.RestoreTableMetadata]. The
+   * [RestoreTableMetadata][google.bigtable.admin.v2.RestoreTableMetadata]. The
    * [response][google.longrunning.Operation.response] type is
    * [Table][google.bigtable.admin.v2.Table], if successful.
    *
@@ -4104,7 +4104,7 @@ public final OperationFuture<Table, RestoreTableMetadata> restoreTableAsync(
    * Create a new table by restoring from a completed backup. The returned table [long-running
    * operation][google.longrunning.Operation] can be used to track the progress of the operation,
    * and to cancel it. The [metadata][google.longrunning.Operation.metadata] field type is
-   * [RestoreTableMetadata][google.bigtable.admin.RestoreTableMetadata]. The
+   * [RestoreTableMetadata][google.bigtable.admin.v2.RestoreTableMetadata]. The
    * [response][google.longrunning.Operation.response] type is
    * [Table][google.bigtable.admin.v2.Table], if successful.
    *
@@ -4140,7 +4140,7 @@ public final OperationFuture<Table, RestoreTableMetadata> restoreTableAsync(
    * Create a new table by restoring from a completed backup. The returned table [long-running
    * operation][google.longrunning.Operation] can be used to track the progress of the operation,
    * and to cancel it. The [metadata][google.longrunning.Operation.metadata] field type is
-   * [RestoreTableMetadata][google.bigtable.admin.RestoreTableMetadata]. The
+   * [RestoreTableMetadata][google.bigtable.admin.v2.RestoreTableMetadata]. The
    * [response][google.longrunning.Operation.response] type is
    * [Table][google.bigtable.admin.v2.Table], if successful.
    *
@@ -4197,7 +4197,7 @@ public final UnaryCallable<RestoreTableRequest, Operation> restoreTableCallable(
    * }</pre>
    *
    * @param parent Required. The name of the destination cluster that will contain the backup copy.
-   *     The cluster must already exists. Values are of the form:
+   *     The cluster must already exist. Values are of the form:
    *     `projects/{project}/instances/{instance}/clusters/{cluster}`.
    * @param backupId Required. The id of the new backup. The `backup_id` along with `parent` are
    *     combined as {parent}/backups/{backup_id} to create the full backup name, of the form:
@@ -4255,7 +4255,7 @@ public final OperationFuture<Backup, CopyBackupMetadata> copyBackupAsync(
    * }</pre>
    *
    * @param parent Required. The name of the destination cluster that will contain the backup copy.
-   *     The cluster must already exists. Values are of the form:
+   *     The cluster must already exist. Values are of the form:
    *     `projects/{project}/instances/{instance}/clusters/{cluster}`.
    * @param backupId Required. The id of the new backup. The `backup_id` along with `parent` are
    *     combined as {parent}/backups/{backup_id} to create the full backup name, of the form:
@@ -4312,7 +4312,7 @@ public final OperationFuture<Backup, CopyBackupMetadata> copyBackupAsync(
    * }</pre>
    *
    * @param parent Required. The name of the destination cluster that will contain the backup copy.
-   *     The cluster must already exists. Values are of the form:
+   *     The cluster must already exist. Values are of the form:
    *     `projects/{project}/instances/{instance}/clusters/{cluster}`.
    * @param backupId Required. The id of the new backup. The `backup_id` along with `parent` are
    *     combined as {parent}/backups/{backup_id} to create the full backup name, of the form:
@@ -4370,7 +4370,7 @@ public final OperationFuture<Backup, CopyBackupMetadata> copyBackupAsync(
    * }</pre>
    *
    * @param parent Required. The name of the destination cluster that will contain the backup copy.
-   *     The cluster must already exists. Values are of the form:
+   *     The cluster must already exist. Values are of the form:
    *     `projects/{project}/instances/{instance}/clusters/{cluster}`.
    * @param backupId Required. The id of the new backup. The `backup_id` along with `parent` are
    *     combined as {parent}/backups/{backup_id} to create the full backup name, of the form:

diff --git a/.../main/java/com/google/cloud/bigtable/admin/v2/stub/BigtableInstanceAdminStubSettings.java b/.../main/java/com/google/cloud/bigtable/admin/v2/stub/BigtableInstanceAdminStubSettings.java
@@ -21,6 +21,7 @@
 
 import com.google.api.core.ApiFunction;
 import com.google.api.core.ApiFuture;
+import com.google.api.core.ObsoleteApi;
 import com.google.api.gax.core.GaxProperties;
 import com.google.api.gax.core.GoogleCredentialsProvider;
 import com.google.api.gax.core.InstantiatingExecutorProvider;
@@ -467,6 +468,7 @@ public static InstantiatingExecutorProvider.Builder defaultExecutorProviderBuild
   }
 
   /** Returns the default service endpoint. */
+  @ObsoleteApi("Use getEndpoint() instead")
   public static String getDefaultEndpoint() {
     return "bigtableadmin.googleapis.com:443";
   }

diff --git a/...src/main/java/com/google/cloud/bigtable/admin/v2/stub/BigtableTableAdminStubSettings.java b/...src/main/java/com/google/cloud/bigtable/admin/v2/stub/BigtableTableAdminStubSettings.java
@@ -23,6 +23,7 @@
 
 import com.google.api.core.ApiFunction;
 import com.google.api.core.ApiFuture;
+import com.google.api.core.ObsoleteApi;
 import com.google.api.gax.core.GaxProperties;
 import com.google.api.gax.core.GoogleCredentialsProvider;
 import com.google.api.gax.core.InstantiatingExecutorProvider;
@@ -683,6 +684,7 @@ public static InstantiatingExecutorProvider.Builder defaultExecutorProviderBuild
   }
 
   /** Returns the default service endpoint. */
+  @ObsoleteApi("Use getEndpoint() instead")
   public static String getDefaultEndpoint() {
     return "bigtableadmin.googleapis.com:443";
   }

diff --git a/...d-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/stub/BigtableStubSettings.java b/...d-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/stub/BigtableStubSettings.java
@@ -18,6 +18,7 @@
 
 import com.google.api.core.ApiFunction;
 import com.google.api.core.InternalApi;
+import com.google.api.core.ObsoleteApi;
 import com.google.api.gax.core.GaxProperties;
 import com.google.api.gax.core.GoogleCredentialsProvider;
 import com.google.api.gax.core.InstantiatingExecutorProvider;
@@ -180,6 +181,7 @@ public static InstantiatingExecutorProvider.Builder defaultExecutorProviderBuild
   }
 
   /** Returns the default service endpoint. */
+  @ObsoleteApi("Use getEndpoint() instead")
   public static String getDefaultEndpoint() {
     return "bigtable.googleapis.com:443";
   }

diff --git a/...in/resources/META-INF/native-image/com.google.cloud.bigtable.admin.v2/reflect-config.json b/...in/resources/META-INF/native-image/com.google.cloud.bigtable.admin.v2/reflect-config.json
@@ -593,6 +593,15 @@
     "allDeclaredClasses": true,
     "allPublicClasses": true
   },
+  {
+    "name": "com.google.bigtable.admin.v2.Backup$BackupType",
+    "queryAllDeclaredConstructors": true,
+    "queryAllPublicConstructors": true,
+    "queryAllDeclaredMethods": true,
+    "allPublicMethods": true,
+    "allDeclaredClasses": true,
+    "allPublicClasses": true
+  },
   {
     "name": "com.google.bigtable.admin.v2.Backup$Builder",
     "queryAllDeclaredConstructors": true,

diff --git a/...le/src/test/java/com/google/cloud/bigtable/admin/v2/BaseBigtableTableAdminClientTest.java b/...le/src/test/java/com/google/cloud/bigtable/admin/v2/BaseBigtableTableAdminClientTest.java
@@ -2096,6 +2096,7 @@ public void createBackupTest() throws Exception {
             .setEndTime(Timestamp.newBuilder().build())
             .setSizeBytes(-1796325715)
             .setEncryptionInfo(EncryptionInfo.newBuilder().build())
+            .setHotToStandardTime(Timestamp.newBuilder().build())
             .build();
     Operation resultOperation =
         Operation.newBuilder()
@@ -2155,6 +2156,7 @@ public void createBackupTest2() throws Exception {
             .setEndTime(Timestamp.newBuilder().build())
             .setSizeBytes(-1796325715)
             .setEncryptionInfo(EncryptionInfo.newBuilder().build())
+            .setHotToStandardTime(Timestamp.newBuilder().build())
             .build();
     Operation resultOperation =
         Operation.newBuilder()
@@ -2214,6 +2216,7 @@ public void getBackupTest() throws Exception {
             .setEndTime(Timestamp.newBuilder().build())
             .setSizeBytes(-1796325715)
             .setEncryptionInfo(EncryptionInfo.newBuilder().build())
+            .setHotToStandardTime(Timestamp.newBuilder().build())
             .build();
     mockBigtableTableAdmin.addResponse(expectedResponse);
 
@@ -2259,6 +2262,7 @@ public void getBackupTest2() throws Exception {
             .setEndTime(Timestamp.newBuilder().build())
             .setSizeBytes(-1796325715)
             .setEncryptionInfo(EncryptionInfo.newBuilder().build())
+            .setHotToStandardTime(Timestamp.newBuilder().build())
             .build();
     mockBigtableTableAdmin.addResponse(expectedResponse);
 
@@ -2304,6 +2308,7 @@ public void updateBackupTest() throws Exception {
             .setEndTime(Timestamp.newBuilder().build())
             .setSizeBytes(-1796325715)
             .setEncryptionInfo(EncryptionInfo.newBuilder().build())
+            .setHotToStandardTime(Timestamp.newBuilder().build())
             .build();
     mockBigtableTableAdmin.addResponse(expectedResponse);
 
@@ -2569,6 +2574,7 @@ public void copyBackupTest() throws Exception {
             .setEndTime(Timestamp.newBuilder().build())
             .setSizeBytes(-1796325715)
             .setEncryptionInfo(EncryptionInfo.newBuilder().build())
+            .setHotToStandardTime(Timestamp.newBuilder().build())
             .build();
     Operation resultOperation =
         Operation.newBuilder()
@@ -2632,6 +2638,7 @@ public void copyBackupTest2() throws Exception {
             .setEndTime(Timestamp.newBuilder().build())
             .setSizeBytes(-1796325715)
             .setEncryptionInfo(EncryptionInfo.newBuilder().build())
+            .setHotToStandardTime(Timestamp.newBuilder().build())
             .build();
     Operation resultOperation =
         Operation.newBuilder()
@@ -2695,6 +2702,7 @@ public void copyBackupTest3() throws Exception {
             .setEndTime(Timestamp.newBuilder().build())
             .setSizeBytes(-1796325715)
             .setEncryptionInfo(EncryptionInfo.newBuilder().build())
+            .setHotToStandardTime(Timestamp.newBuilder().build())
             .build();
     Operation resultOperation =
         Operation.newBuilder()
@@ -2758,6 +2766,7 @@ public void copyBackupTest4() throws Exception {
             .setEndTime(Timestamp.newBuilder().build())
             .setSizeBytes(-1796325715)
             .setEncryptionInfo(EncryptionInfo.newBuilder().build())
+            .setHotToStandardTime(Timestamp.newBuilder().build())
             .build();
     Operation resultOperation =
         Operation.newBuilder()

diff --git a/...-bigtable-admin-v2/src/main/java/com/google/bigtable/admin/v2/BigtableTableAdminGrpc.java b/...-bigtable-admin-v2/src/main/java/com/google/bigtable/admin/v2/BigtableTableAdminGrpc.java
@@ -1834,7 +1834,7 @@ default void listBackups(
      * returned table [long-running operation][google.longrunning.Operation] can
      * be used to track the progress of the operation, and to cancel it.  The
      * [metadata][google.longrunning.Operation.metadata] field type is
-     * [RestoreTableMetadata][google.bigtable.admin.RestoreTableMetadata].  The
+     * [RestoreTableMetadata][google.bigtable.admin.v2.RestoreTableMetadata].  The
      * [response][google.longrunning.Operation.response] type is
      * [Table][google.bigtable.admin.v2.Table], if successful.
      * </pre>
@@ -2402,7 +2402,7 @@ public void listBackups(
      * returned table [long-running operation][google.longrunning.Operation] can
      * be used to track the progress of the operation, and to cancel it.  The
      * [metadata][google.longrunning.Operation.metadata] field type is
-     * [RestoreTableMetadata][google.bigtable.admin.RestoreTableMetadata].  The
+     * [RestoreTableMetadata][google.bigtable.admin.v2.RestoreTableMetadata].  The
      * [response][google.longrunning.Operation.response] type is
      * [Table][google.bigtable.admin.v2.Table], if successful.
      * </pre>
@@ -2887,7 +2887,7 @@ public com.google.bigtable.admin.v2.ListBackupsResponse listBackups(
      * returned table [long-running operation][google.longrunning.Operation] can
      * be used to track the progress of the operation, and to cancel it.  The
      * [metadata][google.longrunning.Operation.metadata] field type is
-     * [RestoreTableMetadata][google.bigtable.admin.RestoreTableMetadata].  The
+     * [RestoreTableMetadata][google.bigtable.admin.v2.RestoreTableMetadata].  The
      * [response][google.longrunning.Operation.response] type is
      * [Table][google.bigtable.admin.v2.Table], if successful.
      * </pre>
@@ -3364,7 +3364,7 @@ protected BigtableTableAdminFutureStub build(
      * returned table [long-running operation][google.longrunning.Operation] can
      * be used to track the progress of the operation, and to cancel it.  The
      * [metadata][google.longrunning.Operation.metadata] field type is
-     * [RestoreTableMetadata][google.bigtable.admin.RestoreTableMetadata].  The
+     * [RestoreTableMetadata][google.bigtable.admin.v2.RestoreTableMetadata].  The
      * [response][google.longrunning.Operation.response] type is
      * [Table][google.bigtable.admin.v2.Table], if successful.
      * </pre>