Skip to content

Commit

Permalink
feat: add context manager support in client (#440)
Browse files Browse the repository at this point in the history
- [ ] Regenerate this pull request now.

chore: fix docstring for first attribute of protos

committer: @busunkim96
PiperOrigin-RevId: 401271153

Source-Link: googleapis/googleapis@787f8c9

Source-Link: googleapis/googleapis-gen@81decff
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiODFkZWNmZmU5ZmM3MjM5NmE4MTUzZTc1NmQxZDY3YTZlZWNmZDYyMCJ9
  • Loading branch information
gcf-owl-bot[bot] committed Oct 8, 2021
1 parent 27e6a85 commit a3d2cf1
Show file tree
Hide file tree
Showing 23 changed files with 306 additions and 15 deletions.
Expand Up @@ -1931,6 +1931,12 @@ async def test_iam_permissions(
# Done; return the response.
return response

async def __aenter__(self):
return self

async def __aexit__(self, exc_type, exc, tb):
await self.transport.close()


try:
DEFAULT_CLIENT_INFO = gapic_v1.client_info.ClientInfo(
Expand Down
Expand Up @@ -408,10 +408,7 @@ def __init__(
client_cert_source_for_mtls=client_cert_source_func,
quota_project_id=client_options.quota_project_id,
client_info=client_info,
always_use_jwt_access=(
Transport == type(self).get_transport_class("grpc")
or Transport == type(self).get_transport_class("grpc_asyncio")
),
always_use_jwt_access=True,
)

def create_instance(
Expand Down Expand Up @@ -2041,6 +2038,19 @@ def test_iam_permissions(
# Done; return the response.
return response

def __enter__(self):
return self

def __exit__(self, type, value, traceback):
"""Releases underlying transport's resources.
.. warning::
ONLY use as a context manager if the transport is NOT shared
with other clients! Exiting the with block will CLOSE the transport
and may cause errors in other clients!
"""
self.transport.close()


try:
DEFAULT_CLIENT_INFO = gapic_v1.client_info.ClientInfo(
Expand Down
Expand Up @@ -371,6 +371,15 @@ def _prep_wrapped_messages(self, client_info):
),
}

def close(self):
"""Closes resources associated with the transport.
.. warning::
Only call this method if the transport is NOT shared
with other clients - this may cause errors in other clients!
"""
raise NotImplementedError()

@property
def operations_client(self) -> operations_v1.OperationsClient:
"""Return the client designed to process long-running operations."""
Expand Down
Expand Up @@ -770,5 +770,8 @@ def test_iam_permissions(
)
return self._stubs["test_iam_permissions"]

def close(self):
self.grpc_channel.close()


__all__ = ("BigtableInstanceAdminGrpcTransport",)
Expand Up @@ -796,5 +796,8 @@ def test_iam_permissions(
)
return self._stubs["test_iam_permissions"]

def close(self):
return self.grpc_channel.close()


__all__ = ("BigtableInstanceAdminGrpcAsyncIOTransport",)
Expand Up @@ -2271,6 +2271,12 @@ async def test_iam_permissions(
# Done; return the response.
return response

async def __aenter__(self):
return self

async def __aexit__(self, exc_type, exc, tb):
await self.transport.close()


try:
DEFAULT_CLIENT_INFO = gapic_v1.client_info.ClientInfo(
Expand Down
Expand Up @@ -446,10 +446,7 @@ def __init__(
client_cert_source_for_mtls=client_cert_source_func,
quota_project_id=client_options.quota_project_id,
client_info=client_info,
always_use_jwt_access=(
Transport == type(self).get_transport_class("grpc")
or Transport == type(self).get_transport_class("grpc_asyncio")
),
always_use_jwt_access=True,
)

def create_table(
Expand Down Expand Up @@ -2443,6 +2440,19 @@ def test_iam_permissions(
# Done; return the response.
return response

def __enter__(self):
return self

def __exit__(self, type, value, traceback):
"""Releases underlying transport's resources.
.. warning::
ONLY use as a context manager if the transport is NOT shared
with other clients! Exiting the with block will CLOSE the transport
and may cause errors in other clients!
"""
self.transport.close()


try:
DEFAULT_CLIENT_INFO = gapic_v1.client_info.ClientInfo(
Expand Down
Expand Up @@ -360,6 +360,15 @@ def _prep_wrapped_messages(self, client_info):
),
}

def close(self):
"""Closes resources associated with the transport.
.. warning::
Only call this method if the transport is NOT shared
with other clients - this may cause errors in other clients!
"""
raise NotImplementedError()

@property
def operations_client(self) -> operations_v1.OperationsClient:
"""Return the client designed to process long-running operations."""
Expand Down
Expand Up @@ -920,5 +920,8 @@ def test_iam_permissions(
)
return self._stubs["test_iam_permissions"]

def close(self):
self.grpc_channel.close()


__all__ = ("BigtableTableAdminGrpcTransport",)
Expand Up @@ -942,5 +942,8 @@ def test_iam_permissions(
)
return self._stubs["test_iam_permissions"]

def close(self):
return self.grpc_channel.close()


__all__ = ("BigtableTableAdminGrpcAsyncIOTransport",)
23 changes: 22 additions & 1 deletion google/cloud/bigtable_admin_v2/types/bigtable_instance_admin.py
Expand Up @@ -51,6 +51,7 @@

class CreateInstanceRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.CreateInstance.
Attributes:
parent (str):
Required. The unique name of the project in which to create
Expand Down Expand Up @@ -82,6 +83,7 @@ class CreateInstanceRequest(proto.Message):

class GetInstanceRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.GetInstance.
Attributes:
name (str):
Required. The unique name of the requested instance. Values
Expand All @@ -93,6 +95,7 @@ class GetInstanceRequest(proto.Message):

class ListInstancesRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.ListInstances.
Attributes:
parent (str):
Required. The unique name of the project for which a list of
Expand All @@ -108,6 +111,7 @@ class ListInstancesRequest(proto.Message):

class ListInstancesResponse(proto.Message):
r"""Response message for BigtableInstanceAdmin.ListInstances.
Attributes:
instances (Sequence[google.cloud.bigtable_admin_v2.types.Instance]):
The list of requested instances.
Expand Down Expand Up @@ -155,6 +159,7 @@ class PartialUpdateInstanceRequest(proto.Message):

class DeleteInstanceRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.DeleteInstance.
Attributes:
name (str):
Required. The unique name of the instance to be deleted.
Expand All @@ -167,6 +172,7 @@ class DeleteInstanceRequest(proto.Message):

class CreateClusterRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.CreateCluster.
Attributes:
parent (str):
Required. The unique name of the instance in which to create
Expand All @@ -189,6 +195,7 @@ class CreateClusterRequest(proto.Message):

class GetClusterRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.GetCluster.
Attributes:
name (str):
Required. The unique name of the requested cluster. Values
Expand All @@ -201,6 +208,7 @@ class GetClusterRequest(proto.Message):

class ListClustersRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.ListClusters.
Attributes:
parent (str):
Required. The unique name of the instance for which a list
Expand All @@ -218,6 +226,7 @@ class ListClustersRequest(proto.Message):

class ListClustersResponse(proto.Message):
r"""Response message for BigtableInstanceAdmin.ListClusters.
Attributes:
clusters (Sequence[google.cloud.bigtable_admin_v2.types.Cluster]):
The list of requested clusters.
Expand Down Expand Up @@ -245,6 +254,7 @@ def raw_page(self):

class DeleteClusterRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.DeleteCluster.
Attributes:
name (str):
Required. The unique name of the cluster to be deleted.
Expand All @@ -257,6 +267,7 @@ class DeleteClusterRequest(proto.Message):

class CreateInstanceMetadata(proto.Message):
r"""The metadata for the Operation returned by CreateInstance.
Attributes:
original_request (google.cloud.bigtable_admin_v2.types.CreateInstanceRequest):
The request that prompted the initiation of
Expand All @@ -280,6 +291,7 @@ class CreateInstanceMetadata(proto.Message):

class UpdateInstanceMetadata(proto.Message):
r"""The metadata for the Operation returned by UpdateInstance.
Attributes:
original_request (google.cloud.bigtable_admin_v2.types.PartialUpdateInstanceRequest):
The request that prompted the initiation of
Expand All @@ -303,6 +315,7 @@ class UpdateInstanceMetadata(proto.Message):

class CreateClusterMetadata(proto.Message):
r"""The metadata for the Operation returned by CreateCluster.
Attributes:
original_request (google.cloud.bigtable_admin_v2.types.CreateClusterRequest):
The request that prompted the initiation of
Expand All @@ -326,6 +339,7 @@ class CreateClusterMetadata(proto.Message):

class UpdateClusterMetadata(proto.Message):
r"""The metadata for the Operation returned by UpdateCluster.
Attributes:
original_request (google.cloud.bigtable_admin_v2.types.Cluster):
The request that prompted the initiation of
Expand All @@ -349,6 +363,7 @@ class UpdateClusterMetadata(proto.Message):

class CreateAppProfileRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.CreateAppProfile.
Attributes:
parent (str):
Required. The unique name of the instance in which to create
Expand All @@ -375,6 +390,7 @@ class CreateAppProfileRequest(proto.Message):

class GetAppProfileRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.GetAppProfile.
Attributes:
name (str):
Required. The unique name of the requested app profile.
Expand All @@ -387,6 +403,7 @@ class GetAppProfileRequest(proto.Message):

class ListAppProfilesRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.ListAppProfiles.
Attributes:
parent (str):
Required. The unique name of the instance for which a list
Expand Down Expand Up @@ -418,6 +435,7 @@ class ListAppProfilesRequest(proto.Message):

class ListAppProfilesResponse(proto.Message):
r"""Response message for BigtableInstanceAdmin.ListAppProfiles.
Attributes:
app_profiles (Sequence[google.cloud.bigtable_admin_v2.types.AppProfile]):
The list of requested app profiles.
Expand Down Expand Up @@ -446,6 +464,7 @@ def raw_page(self):

class UpdateAppProfileRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.UpdateAppProfile.
Attributes:
app_profile (google.cloud.bigtable_admin_v2.types.AppProfile):
Required. The app profile which will
Expand All @@ -468,6 +487,7 @@ class UpdateAppProfileRequest(proto.Message):

class DeleteAppProfileRequest(proto.Message):
r"""Request message for BigtableInstanceAdmin.DeleteAppProfile.
Attributes:
name (str):
Required. The unique name of the app profile to be deleted.
Expand All @@ -483,7 +503,8 @@ class DeleteAppProfileRequest(proto.Message):


class UpdateAppProfileMetadata(proto.Message):
r"""The metadata for the Operation returned by UpdateAppProfile. """
r"""The metadata for the Operation returned by UpdateAppProfile.
"""


__all__ = tuple(sorted(__protobuf__.manifest))
2 changes: 2 additions & 0 deletions google/cloud/bigtable_admin_v2/types/bigtable_table_admin.py
Expand Up @@ -183,6 +183,7 @@ class CreateTableRequest(proto.Message):

class Split(proto.Message):
r"""An initial split point for a newly created table.
Attributes:
key (bytes):
Row key to use as an initial tablet boundary.
Expand Down Expand Up @@ -357,6 +358,7 @@ class ModifyColumnFamiliesRequest(proto.Message):

class Modification(proto.Message):
r"""A create, update, or delete of a particular column family.
Attributes:
id (str):
The ID of the column family to be modified.
Expand Down
6 changes: 6 additions & 0 deletions google/cloud/bigtable_admin_v2/types/table.py
Expand Up @@ -44,6 +44,7 @@ class RestoreSourceType(proto.Enum):

class RestoreInfo(proto.Message):
r"""Information about a table restore.
Attributes:
source_type (google.cloud.bigtable_admin_v2.types.RestoreSourceType):
The type of the restore source.
Expand Down Expand Up @@ -111,6 +112,7 @@ class View(proto.Enum):

class ClusterState(proto.Message):
r"""The state of a table's data in a particular cluster.
Attributes:
replication_state (google.cloud.bigtable_admin_v2.types.Table.ClusterState.ReplicationState):
Output only. The state of replication for the
Expand Down Expand Up @@ -193,6 +195,7 @@ class GcRule(proto.Message):

class Intersection(proto.Message):
r"""A GcRule which deletes cells matching all of the given rules.
Attributes:
rules (Sequence[google.cloud.bigtable_admin_v2.types.GcRule]):
Only delete cells which would be deleted by every element of
Expand All @@ -203,6 +206,7 @@ class Intersection(proto.Message):

class Union(proto.Message):
r"""A GcRule which deletes cells matching any of the given rules.
Attributes:
rules (Sequence[google.cloud.bigtable_admin_v2.types.GcRule]):
Delete cells which would be deleted by any element of
Expand Down Expand Up @@ -310,6 +314,7 @@ class State(proto.Enum):

class Backup(proto.Message):
r"""A backup of a Cloud Bigtable table.
Attributes:
name (str):
Output only. A globally unique identifier for the backup
Expand Down Expand Up @@ -369,6 +374,7 @@ class State(proto.Enum):

class BackupInfo(proto.Message):
r"""Information about a backup.
Attributes:
backup (str):
Output only. Name of the backup.
Expand Down

0 comments on commit a3d2cf1

Please sign in to comment.