From d8bfe0ba23dcd103fe91bd8d3215a87c5f45a041 Mon Sep 17 00:00:00 2001 From: Mattie Fu Date: Thu, 5 Nov 2026 16:27:40 +0000 Subject: [PATCH 1/2] chore(bigtable): deprecate legacy handwritten Table and Backup admin methods Add DeprecationWarning and Sphinx deprecated directives to legacy handwritten admin methods on Table and Backup in favor of BigtableTableAdminClient. --- .../google/cloud/bigtable/backup.py | 130 +++++++++++++- .../google/cloud/bigtable/table.py | 164 ++++++++++++++++++ 2 files changed, 292 insertions(+), 2 deletions(-) diff --git a/packages/google-cloud-bigtable/google/cloud/bigtable/backup.py b/packages/google-cloud-bigtable/google/cloud/bigtable/backup.py index ef3c97543818..a4ecd4f2e76e 100644 --- a/packages/google-cloud-bigtable/google/cloud/bigtable/backup.py +++ b/packages/google-cloud-bigtable/google/cloud/bigtable/backup.py @@ -15,6 +15,7 @@ """A user-friendly wrapper for a Google Cloud Bigtable Backup.""" import re +import warnings from google.cloud._helpers import _datetime_to_pb_timestamp # type: ignore from google.cloud.exceptions import NotFound # type: ignore @@ -42,6 +43,9 @@ class Backup(object): """Representation of a Google Cloud Bigtable Backup. + .. deprecated:: 2.30.0 + Use :class:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient` instead. + A :class: `Backup` can be used to: * :meth:`create` the backup @@ -301,6 +305,11 @@ def __ne__(self, other): def create(self, cluster_id=None): """Creates this backup within its instance. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.create_backup` + instead. + :type cluster_id: str :param cluster_id: (Optional) The ID of the Cluster for the newly created Backup. @@ -312,6 +321,13 @@ def create(self, cluster_id=None): :raises BadRequest: if the `table` or `expire_time` values are invalid, or `expire_time` is not set """ + warnings.warn( + "Backup.create is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "create_backup instead.", + DeprecationWarning, + stacklevel=2, + ) if not self._expire_time: raise ValueError('"expire_time" parameter must be set') # TODO: Consider implementing a method that sets a default value of @@ -342,6 +358,11 @@ def create(self, cluster_id=None): def get(self): """Retrieves metadata of a pending or completed Backup. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.get_backup` + instead. + :returns: An instance of :class:`~google.cloud.bigtable_admin_v2.types.Backup` @@ -351,6 +372,13 @@ def get(self): due to a retryable error and retry attempts failed. :raises ValueError: If the parameters are invalid. """ + warnings.warn( + "Backup.get is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "get_backup instead.", + DeprecationWarning, + stacklevel=2, + ) api = self._instance._client.table_admin_client try: return api.get_backup(request={"name": self.name}) @@ -358,7 +386,20 @@ def get(self): return None def reload(self): - """Refreshes the stored backup properties.""" + """Refreshes the stored backup properties. + + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.get_backup` + instead. + """ + warnings.warn( + "Backup.reload is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "get_backup instead.", + DeprecationWarning, + stacklevel=2, + ) backup = self.get() self._source_table = backup.source_table self._expire_time = backup._pb.expire_time @@ -370,17 +411,41 @@ def reload(self): def exists(self): """Tests whether this Backup exists. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.get_backup` + instead. + :rtype: bool :returns: True if the Backup exists, else False. """ + warnings.warn( + "Backup.exists is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "get_backup instead.", + DeprecationWarning, + stacklevel=2, + ) return self.get() is not None def update_expire_time(self, new_expire_time): """Update the expire time of this Backup. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.update_backup` + instead. + :type new_expire_time: :class:`datetime.datetime` :param new_expire_time: the new expiration time timestamp """ + warnings.warn( + "Backup.update_expire_time is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "update_backup instead.", + DeprecationWarning, + stacklevel=2, + ) backup_update = table.Backup( name=self.name, expire_time=_datetime_to_pb_timestamp(new_expire_time), @@ -391,7 +456,20 @@ def update_expire_time(self, new_expire_time): self._expire_time = new_expire_time def delete(self): - """Delete this Backup.""" + """Delete this Backup. + + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.delete_backup` + instead. + """ + warnings.warn( + "Backup.delete is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "delete_backup instead.", + DeprecationWarning, + stacklevel=2, + ) self._instance._client.table_admin_client.delete_backup( request={"name": self.name} ) @@ -404,6 +482,11 @@ def restore(self, table_id, instance_id=None): progress of the operation and to cancel it. The ``response`` type is ``Table``, if successful. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.restore_table` + instead. + :type table_id: str :param table_id: The ID of the Table to create and restore to. This Table must not already exist. @@ -424,6 +507,13 @@ def restore(self, table_id, instance_id=None): due to a retryable error and retry attempts failed. :raises: ValueError: If the parameters are invalid. """ + warnings.warn( + "Backup.restore is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "restore_table instead.", + DeprecationWarning, + stacklevel=2, + ) api = self._instance._client.table_admin_client if instance_id: parent = BaseBigtableTableAdminClient.instance_path( @@ -440,9 +530,21 @@ def restore(self, table_id, instance_id=None): def get_iam_policy(self): """Gets the IAM access control policy for this backup. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.get_iam_policy` + instead. + :rtype: :class:`google.cloud.bigtable.policy.Policy` :returns: The current IAM policy of this backup. """ + warnings.warn( + "Backup.get_iam_policy is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "get_iam_policy instead.", + DeprecationWarning, + stacklevel=2, + ) table_api = self._instance._client.table_admin_client response = table_api.get_iam_policy(request={"resource": self.name}) return Policy.from_pb(response) @@ -451,6 +553,11 @@ def set_iam_policy(self, policy): """Sets the IAM access control policy for this backup. Replaces any existing policy. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.set_iam_policy` + instead. + For more information about policy, please see documentation of class `google.cloud.bigtable.policy.Policy` @@ -461,6 +568,13 @@ class `google.cloud.bigtable.policy.Policy` :rtype: :class:`google.cloud.bigtable.policy.Policy` :returns: The current IAM policy of this backup. """ + warnings.warn( + "Backup.set_iam_policy is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "set_iam_policy instead.", + DeprecationWarning, + stacklevel=2, + ) table_api = self._instance._client.table_admin_client response = table_api.set_iam_policy( request={"resource": self.name, "policy": policy.to_pb()} @@ -471,6 +585,11 @@ def test_iam_permissions(self, permissions): """Tests whether the caller has the given permissions for this backup. Returns the permissions that the caller has. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.test_iam_permissions` + instead. + :type permissions: list :param permissions: The set of permissions to check for the ``resource``. Permissions with wildcards (such as '*' @@ -483,6 +602,13 @@ def test_iam_permissions(self, permissions): :rtype: list :returns: A List(string) of permissions allowed on the backup. """ + warnings.warn( + "Backup.test_iam_permissions is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "test_iam_permissions instead.", + DeprecationWarning, + stacklevel=2, + ) table_api = self._instance._client.table_admin_client response = table_api.test_iam_permissions( request={"resource": self.name, "permissions": permissions} diff --git a/packages/google-cloud-bigtable/google/cloud/bigtable/table.py b/packages/google-cloud-bigtable/google/cloud/bigtable/table.py index d86b7acf2ee9..06f7626c4345 100644 --- a/packages/google-cloud-bigtable/google/cloud/bigtable/table.py +++ b/packages/google-cloud-bigtable/google/cloud/bigtable/table.py @@ -181,6 +181,11 @@ def name(self): def get_iam_policy(self): """Gets the IAM access control policy for this table. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.get_iam_policy` + instead. + For example: .. literalinclude:: snippets_table.py @@ -191,6 +196,13 @@ def get_iam_policy(self): :rtype: :class:`google.cloud.bigtable.policy.Policy` :returns: The current IAM policy of this table. """ + warnings.warn( + "Table.get_iam_policy is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "get_iam_policy instead.", + DeprecationWarning, + stacklevel=2, + ) table_client = self._instance._client.table_admin_client resp = table_client.get_iam_policy(request={"resource": self.name}) return Policy.from_pb(resp) @@ -199,6 +211,11 @@ def set_iam_policy(self, policy): """Sets the IAM access control policy for this table. Replaces any existing policy. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.set_iam_policy` + instead. + For more information about policy, please see documentation of class `google.cloud.bigtable.policy.Policy` @@ -216,6 +233,13 @@ class `google.cloud.bigtable.policy.Policy` :rtype: :class:`google.cloud.bigtable.policy.Policy` :returns: The current IAM policy of this table. """ + warnings.warn( + "Table.set_iam_policy is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "set_iam_policy instead.", + DeprecationWarning, + stacklevel=2, + ) table_client = self._instance._client.table_admin_client resp = table_client.set_iam_policy( request={"resource": self.name, "policy": policy.to_pb()} @@ -226,6 +250,11 @@ def test_iam_permissions(self, permissions): """Tests whether the caller has the given permissions for this table. Returns the permissions that the caller has. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.test_iam_permissions` + instead. + For example: .. literalinclude:: snippets_table.py @@ -245,6 +274,13 @@ def test_iam_permissions(self, permissions): :rtype: list :returns: A List(string) of permissions allowed on the table. """ + warnings.warn( + "Table.test_iam_permissions is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "test_iam_permissions instead.", + DeprecationWarning, + stacklevel=2, + ) table_client = self._instance._client.table_admin_client resp = table_client.test_iam_permissions( request={"resource": self.name, "permissions": permissions} @@ -254,6 +290,10 @@ def test_iam_permissions(self, permissions): def column_family(self, column_family_id, gc_rule=None): """Factory to create a column family associated with this table. + .. deprecated:: 2.30.0 + Use :class:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient` + instead. + For example: .. literalinclude:: snippets_table.py @@ -390,6 +430,11 @@ def __ne__(self, other): def create(self, initial_split_keys=[], column_families={}): """Creates this table. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.create_table` + instead. + For example: .. literalinclude:: snippets_table.py @@ -413,6 +458,13 @@ def create(self, initial_split_keys=[], column_families={}): the column_id str and the value is a :class:`GarbageCollectionRule` """ + warnings.warn( + "Table.create is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "create_table instead.", + DeprecationWarning, + stacklevel=2, + ) table_client = self._instance._client.table_admin_client instance_name = self._instance.name @@ -437,6 +489,11 @@ def create(self, initial_split_keys=[], column_families={}): def exists(self): """Check whether the table exists. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.get_table` + instead. + For example: .. literalinclude:: snippets_table.py @@ -447,6 +504,13 @@ def exists(self): :rtype: bool :returns: True if the table exists, else False. """ + warnings.warn( + "Table.exists is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "get_table instead.", + DeprecationWarning, + stacklevel=2, + ) table_client = self._instance._client.table_admin_client try: table_client.get_table(request={"name": self.name, "view": VIEW_NAME_ONLY}) @@ -457,6 +521,11 @@ def exists(self): def delete(self): """Delete this table. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.delete_table` + instead. + For example: .. literalinclude:: snippets_table.py @@ -464,12 +533,24 @@ def delete(self): :end-before: [END bigtable_api_delete_table] :dedent: 4 """ + warnings.warn( + "Table.delete is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "delete_table instead.", + DeprecationWarning, + stacklevel=2, + ) table_client = self._instance._client.table_admin_client table_client.delete_table(request={"name": self.name}) def list_column_families(self): """List the column families owned by this table. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.get_table` + instead. + For example: .. literalinclude:: snippets_table.py @@ -485,6 +566,13 @@ def list_column_families(self): family name from the response does not agree with the computed name from the column family ID. """ + warnings.warn( + "Table.list_column_families is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "get_table instead.", + DeprecationWarning, + stacklevel=2, + ) table_client = self._instance._client.table_admin_client table_pb = table_client.get_table(request={"name": self.name}) @@ -498,6 +586,11 @@ def list_column_families(self): def get_cluster_states(self): """List the cluster states owned by this table. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.get_table` + instead. + For example: .. literalinclude:: snippets_table.py @@ -510,6 +603,13 @@ def get_cluster_states(self): Keys are cluster ids and values are :class: 'ClusterState' instances. """ + warnings.warn( + "Table.get_cluster_states is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "get_table instead.", + DeprecationWarning, + stacklevel=2, + ) REPLICATION_VIEW = enums.Table.View.REPLICATION_VIEW table_client = self._instance._client.table_admin_client @@ -525,6 +625,11 @@ def get_cluster_states(self): def get_encryption_info(self): """List the encryption info for each cluster owned by this table. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.get_table` + instead. + Gets the current encryption info for the table across all of the clusters. The returned dict will be keyed by cluster id and contain a status for all of the keys in use. @@ -533,6 +638,13 @@ def get_encryption_info(self): :returns: Dictionary of encryption info for this table. Keys are cluster ids and values are tuples of :class:`google.cloud.bigtable.encryption.EncryptionInfo` instances. """ + warnings.warn( + "Table.get_encryption_info is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "get_table instead.", + DeprecationWarning, + stacklevel=2, + ) ENCRYPTION_VIEW = enums.Table.View.ENCRYPTION_VIEW table_client = self._instance._client.table_admin_client table_pb = table_client.get_table( @@ -873,6 +985,11 @@ def sample_row_keys(self): def truncate(self, timeout=None): """Truncate the table + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.drop_row_range` + instead. + For example: .. literalinclude:: snippets_table.py @@ -890,6 +1007,13 @@ def truncate(self, timeout=None): due to a retryable error and retry attempts failed. ValueError: If the parameters are invalid. """ + warnings.warn( + "Table.truncate is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "drop_row_range instead.", + DeprecationWarning, + stacklevel=2, + ) client = self._instance._client table_admin_client = client.table_admin_client if timeout: @@ -905,6 +1029,11 @@ def truncate(self, timeout=None): def drop_by_prefix(self, row_key_prefix, timeout=None): """ + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.drop_row_range` + instead. + For example: .. literalinclude:: snippets_table.py @@ -926,6 +1055,13 @@ def drop_by_prefix(self, row_key_prefix, timeout=None): due to a retryable error and retry attempts failed. ValueError: If the parameters are invalid. """ + warnings.warn( + "Table.drop_by_prefix is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "drop_row_range instead.", + DeprecationWarning, + stacklevel=2, + ) client = self._instance._client table_admin_client = client.table_admin_client if timeout: @@ -977,6 +1113,10 @@ def mutations_batcher( def backup(self, backup_id, cluster_id=None, expire_time=None): """Factory to create a Backup linked to this Table. + .. deprecated:: 2.30.0 + Use :class:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient` + instead. + :type backup_id: str :param backup_id: The ID of the Backup to be created. @@ -1001,6 +1141,11 @@ def backup(self, backup_id, cluster_id=None, expire_time=None): def list_backups(self, cluster_id=None, filter_=None, order_by=None, page_size=0): """List Backups for this Table. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.list_backups` + instead. + :type cluster_id: str :param cluster_id: (Optional) Specifies a single cluster to list Backups from. If none is specified, the returned list @@ -1078,6 +1223,13 @@ def list_backups(self, cluster_id=None, filter_=None, order_by=None, page_size=0 :raises: :class:`ValueError ` if one of the returned Backups' name is not of the expected format. """ + warnings.warn( + "Table.list_backups is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "list_backups instead.", + DeprecationWarning, + stacklevel=2, + ) cluster_id = cluster_id or "-" backups_filter = "source_table:{}".format(self.name) @@ -1111,6 +1263,11 @@ def restore(self, new_table_id, cluster_id=None, backup_id=None, backup_name=Non can be used to track the progress of the operation and to cancel it. The ``response`` type is ``Table``, if successful. + .. deprecated:: 2.30.0 + Use + :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.restore_table` + instead. + :type new_table_id: str :param new_table_id: The ID of the Table to create and restore to. This Table must not already exist. @@ -1141,6 +1298,13 @@ def restore(self, new_table_id, cluster_id=None, backup_id=None, backup_name=Non due to a retryable error and retry attempts failed. :raises: ValueError: If the parameters are invalid. """ + warnings.warn( + "Table.restore is deprecated. Use " + "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." + "restore_table instead.", + DeprecationWarning, + stacklevel=2, + ) api = self._instance._client.table_admin_client if not backup_name: backup_name = BaseBigtableTableAdminClient.backup_path( From bc4e03a134daa6e709b6b2e7ba195c68219a3b38 Mon Sep 17 00:00:00 2001 From: Mattie Fu Date: Thu, 5 Nov 2026 19:51:28 +0000 Subject: [PATCH 2/2] docs(bigtable): remove runtime DeprecationWarning from Table and Backup admin methods Keep only the Sphinx .. deprecated:: 2.30.0 docstring notices without emitting runtime warnings. --- .../google/cloud/bigtable/backup.py | 71 --------------- .../google/cloud/bigtable/table.py | 91 ------------------- 2 files changed, 162 deletions(-) diff --git a/packages/google-cloud-bigtable/google/cloud/bigtable/backup.py b/packages/google-cloud-bigtable/google/cloud/bigtable/backup.py index a4ecd4f2e76e..99ad46517723 100644 --- a/packages/google-cloud-bigtable/google/cloud/bigtable/backup.py +++ b/packages/google-cloud-bigtable/google/cloud/bigtable/backup.py @@ -15,7 +15,6 @@ """A user-friendly wrapper for a Google Cloud Bigtable Backup.""" import re -import warnings from google.cloud._helpers import _datetime_to_pb_timestamp # type: ignore from google.cloud.exceptions import NotFound # type: ignore @@ -321,13 +320,6 @@ def create(self, cluster_id=None): :raises BadRequest: if the `table` or `expire_time` values are invalid, or `expire_time` is not set """ - warnings.warn( - "Backup.create is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "create_backup instead.", - DeprecationWarning, - stacklevel=2, - ) if not self._expire_time: raise ValueError('"expire_time" parameter must be set') # TODO: Consider implementing a method that sets a default value of @@ -372,13 +364,6 @@ def get(self): due to a retryable error and retry attempts failed. :raises ValueError: If the parameters are invalid. """ - warnings.warn( - "Backup.get is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "get_backup instead.", - DeprecationWarning, - stacklevel=2, - ) api = self._instance._client.table_admin_client try: return api.get_backup(request={"name": self.name}) @@ -393,13 +378,6 @@ def reload(self): :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.get_backup` instead. """ - warnings.warn( - "Backup.reload is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "get_backup instead.", - DeprecationWarning, - stacklevel=2, - ) backup = self.get() self._source_table = backup.source_table self._expire_time = backup._pb.expire_time @@ -419,13 +397,6 @@ def exists(self): :rtype: bool :returns: True if the Backup exists, else False. """ - warnings.warn( - "Backup.exists is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "get_backup instead.", - DeprecationWarning, - stacklevel=2, - ) return self.get() is not None def update_expire_time(self, new_expire_time): @@ -439,13 +410,6 @@ def update_expire_time(self, new_expire_time): :type new_expire_time: :class:`datetime.datetime` :param new_expire_time: the new expiration time timestamp """ - warnings.warn( - "Backup.update_expire_time is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "update_backup instead.", - DeprecationWarning, - stacklevel=2, - ) backup_update = table.Backup( name=self.name, expire_time=_datetime_to_pb_timestamp(new_expire_time), @@ -463,13 +427,6 @@ def delete(self): :meth:`google.cloud.bigtable_admin_v2.BigtableTableAdminClient.delete_backup` instead. """ - warnings.warn( - "Backup.delete is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "delete_backup instead.", - DeprecationWarning, - stacklevel=2, - ) self._instance._client.table_admin_client.delete_backup( request={"name": self.name} ) @@ -507,13 +464,6 @@ def restore(self, table_id, instance_id=None): due to a retryable error and retry attempts failed. :raises: ValueError: If the parameters are invalid. """ - warnings.warn( - "Backup.restore is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "restore_table instead.", - DeprecationWarning, - stacklevel=2, - ) api = self._instance._client.table_admin_client if instance_id: parent = BaseBigtableTableAdminClient.instance_path( @@ -538,13 +488,6 @@ def get_iam_policy(self): :rtype: :class:`google.cloud.bigtable.policy.Policy` :returns: The current IAM policy of this backup. """ - warnings.warn( - "Backup.get_iam_policy is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "get_iam_policy instead.", - DeprecationWarning, - stacklevel=2, - ) table_api = self._instance._client.table_admin_client response = table_api.get_iam_policy(request={"resource": self.name}) return Policy.from_pb(response) @@ -568,13 +511,6 @@ class `google.cloud.bigtable.policy.Policy` :rtype: :class:`google.cloud.bigtable.policy.Policy` :returns: The current IAM policy of this backup. """ - warnings.warn( - "Backup.set_iam_policy is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "set_iam_policy instead.", - DeprecationWarning, - stacklevel=2, - ) table_api = self._instance._client.table_admin_client response = table_api.set_iam_policy( request={"resource": self.name, "policy": policy.to_pb()} @@ -602,13 +538,6 @@ def test_iam_permissions(self, permissions): :rtype: list :returns: A List(string) of permissions allowed on the backup. """ - warnings.warn( - "Backup.test_iam_permissions is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "test_iam_permissions instead.", - DeprecationWarning, - stacklevel=2, - ) table_api = self._instance._client.table_admin_client response = table_api.test_iam_permissions( request={"resource": self.name, "permissions": permissions} diff --git a/packages/google-cloud-bigtable/google/cloud/bigtable/table.py b/packages/google-cloud-bigtable/google/cloud/bigtable/table.py index 06f7626c4345..9be6f3972422 100644 --- a/packages/google-cloud-bigtable/google/cloud/bigtable/table.py +++ b/packages/google-cloud-bigtable/google/cloud/bigtable/table.py @@ -196,13 +196,6 @@ def get_iam_policy(self): :rtype: :class:`google.cloud.bigtable.policy.Policy` :returns: The current IAM policy of this table. """ - warnings.warn( - "Table.get_iam_policy is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "get_iam_policy instead.", - DeprecationWarning, - stacklevel=2, - ) table_client = self._instance._client.table_admin_client resp = table_client.get_iam_policy(request={"resource": self.name}) return Policy.from_pb(resp) @@ -233,13 +226,6 @@ class `google.cloud.bigtable.policy.Policy` :rtype: :class:`google.cloud.bigtable.policy.Policy` :returns: The current IAM policy of this table. """ - warnings.warn( - "Table.set_iam_policy is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "set_iam_policy instead.", - DeprecationWarning, - stacklevel=2, - ) table_client = self._instance._client.table_admin_client resp = table_client.set_iam_policy( request={"resource": self.name, "policy": policy.to_pb()} @@ -274,13 +260,6 @@ def test_iam_permissions(self, permissions): :rtype: list :returns: A List(string) of permissions allowed on the table. """ - warnings.warn( - "Table.test_iam_permissions is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "test_iam_permissions instead.", - DeprecationWarning, - stacklevel=2, - ) table_client = self._instance._client.table_admin_client resp = table_client.test_iam_permissions( request={"resource": self.name, "permissions": permissions} @@ -458,13 +437,6 @@ def create(self, initial_split_keys=[], column_families={}): the column_id str and the value is a :class:`GarbageCollectionRule` """ - warnings.warn( - "Table.create is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "create_table instead.", - DeprecationWarning, - stacklevel=2, - ) table_client = self._instance._client.table_admin_client instance_name = self._instance.name @@ -504,13 +476,6 @@ def exists(self): :rtype: bool :returns: True if the table exists, else False. """ - warnings.warn( - "Table.exists is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "get_table instead.", - DeprecationWarning, - stacklevel=2, - ) table_client = self._instance._client.table_admin_client try: table_client.get_table(request={"name": self.name, "view": VIEW_NAME_ONLY}) @@ -533,13 +498,6 @@ def delete(self): :end-before: [END bigtable_api_delete_table] :dedent: 4 """ - warnings.warn( - "Table.delete is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "delete_table instead.", - DeprecationWarning, - stacklevel=2, - ) table_client = self._instance._client.table_admin_client table_client.delete_table(request={"name": self.name}) @@ -566,13 +524,6 @@ def list_column_families(self): family name from the response does not agree with the computed name from the column family ID. """ - warnings.warn( - "Table.list_column_families is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "get_table instead.", - DeprecationWarning, - stacklevel=2, - ) table_client = self._instance._client.table_admin_client table_pb = table_client.get_table(request={"name": self.name}) @@ -603,13 +554,6 @@ def get_cluster_states(self): Keys are cluster ids and values are :class: 'ClusterState' instances. """ - warnings.warn( - "Table.get_cluster_states is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "get_table instead.", - DeprecationWarning, - stacklevel=2, - ) REPLICATION_VIEW = enums.Table.View.REPLICATION_VIEW table_client = self._instance._client.table_admin_client @@ -638,13 +582,6 @@ def get_encryption_info(self): :returns: Dictionary of encryption info for this table. Keys are cluster ids and values are tuples of :class:`google.cloud.bigtable.encryption.EncryptionInfo` instances. """ - warnings.warn( - "Table.get_encryption_info is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "get_table instead.", - DeprecationWarning, - stacklevel=2, - ) ENCRYPTION_VIEW = enums.Table.View.ENCRYPTION_VIEW table_client = self._instance._client.table_admin_client table_pb = table_client.get_table( @@ -1007,13 +944,6 @@ def truncate(self, timeout=None): due to a retryable error and retry attempts failed. ValueError: If the parameters are invalid. """ - warnings.warn( - "Table.truncate is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "drop_row_range instead.", - DeprecationWarning, - stacklevel=2, - ) client = self._instance._client table_admin_client = client.table_admin_client if timeout: @@ -1055,13 +985,6 @@ def drop_by_prefix(self, row_key_prefix, timeout=None): due to a retryable error and retry attempts failed. ValueError: If the parameters are invalid. """ - warnings.warn( - "Table.drop_by_prefix is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "drop_row_range instead.", - DeprecationWarning, - stacklevel=2, - ) client = self._instance._client table_admin_client = client.table_admin_client if timeout: @@ -1223,13 +1146,6 @@ def list_backups(self, cluster_id=None, filter_=None, order_by=None, page_size=0 :raises: :class:`ValueError ` if one of the returned Backups' name is not of the expected format. """ - warnings.warn( - "Table.list_backups is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "list_backups instead.", - DeprecationWarning, - stacklevel=2, - ) cluster_id = cluster_id or "-" backups_filter = "source_table:{}".format(self.name) @@ -1298,13 +1214,6 @@ def restore(self, new_table_id, cluster_id=None, backup_id=None, backup_name=Non due to a retryable error and retry attempts failed. :raises: ValueError: If the parameters are invalid. """ - warnings.warn( - "Table.restore is deprecated. Use " - "google.cloud.bigtable_admin_v2.BigtableTableAdminClient." - "restore_table instead.", - DeprecationWarning, - stacklevel=2, - ) api = self._instance._client.table_admin_client if not backup_name: backup_name = BaseBigtableTableAdminClient.backup_path(