Merge branch 'main' into document_v3

.github/CODEOWNERS

@@ -5,8 +5,8 @@
 # https://help.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners#codeowners-syntax
 # Note: This file is autogenerated. To make changes to the codeowner team, please update .repo-metadata.json.
 
-# @googleapis/yoshi-python @googleapis/api-bigtable are the default owners for changes in this repo
-*     @googleapis/yoshi-python @googleapis/api-bigtable
+# @googleapis/yoshi-python @googleapis/api-bigtable @googleapis/api-bigtable-partners are the default owners for changes in this repo
+*     @googleapis/yoshi-python @googleapis/api-bigtable @googleapis/api-bigtable-partners
 
-# @googleapis/python-samples-reviewers @googleapis/api-bigtable are the default owners for samples changes
-/samples/   @googleapis/python-samples-reviewers @googleapis/api-bigtable
+# @googleapis/python-samples-reviewers @googleapis/api-bigtable @googleapis/api-bigtable-partners are the default owners for samples changes
+/samples/   @googleapis/python-samples-reviewers @googleapis/api-bigtable @googleapis/api-bigtable-partners

.github/flakybot.yaml

@@ -0,0 +1,15 @@
+# Copyright 2024 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+issuePriority: p2

.repo-metadata.json

@@ -75,6 +75,6 @@
         }
     ],
     "default_version": "v2",
-    "codeowner_team": "@googleapis/api-bigtable",
+    "codeowner_team": "@googleapis/api-bigtable @googleapis/api-bigtable-partners",
     "api_shortname": "bigtable"
 }

google/cloud/bigtable/data/_async/client.py

@@ -86,6 +86,16 @@
 
 
 class BigtableDataClientAsync(ClientWithProject):
+    def __init__(
+        self,
+        *,
+        project: str | None = None,
+        pool_size: int = 3,
+        credentials: google.auth.credentials.Credentials | None = None,
+        client_options: dict[str, Any]
+        | "google.api_core.client_options.ClientOptions"
+        | None = None,
+    ):
         """
         Create a client instance for the Bigtable Data API
 
@@ -112,17 +122,6 @@ class BigtableDataClientAsync(ClientWithProject):
           - RuntimeError if called outside of an async context (no running event loop)
           - ValueError if pool_size is less than 1
         """
-
-    def __init__(
-        self,
-        *,
-        project: str | None = None,
-        pool_size: int = 3,
-        credentials: google.auth.credentials.Credentials | None = None,
-        client_options: dict[str, Any]
-        | "google.api_core.client_options.ClientOptions"
-        | None = None,
-    ):
         # set up transport in registry
         transport_str = f"pooled_grpc_asyncio_{pool_size}"
         transport = PooledBigtableGrpcAsyncIOTransport.with_fixed_size(pool_size)
@@ -987,38 +986,38 @@ async def mutate_row(
         | TABLE_DEFAULT = TABLE_DEFAULT.DEFAULT,
     ):
         """
-         Mutates a row atomically.
+        Mutates a row atomically.
 
-         Cells already present in the row are left unchanged unless explicitly changed
-         by ``mutation``.
+        Cells already present in the row are left unchanged unless explicitly changed
+        by ``mutation``.
 
-         Idempotent operations (i.e, all mutations have an explicit timestamp) will be
-         retried on server failure. Non-idempotent operations will not.
+        Idempotent operations (i.e, all mutations have an explicit timestamp) will be
+        retried on server failure. Non-idempotent operations will not.
 
         Warning: BigtableDataClientAsync is currently experimental, and is not
         yet recommended for production use.
 
-         Args:
-            - row_key: the row to apply mutations to
-            - mutations: the set of mutations to apply to the row
-            - operation_timeout: the time budget for the entire operation, in seconds.
-                Failed requests will be retried within the budget.
-                Defaults to the Table's default_operation_timeout
-            - attempt_timeout: the time budget for an individual network request, in seconds.
-                If it takes longer than this time to complete, the request will be cancelled with
-                a DeadlineExceeded exception, and a retry will be attempted.
-                Defaults to the Table's default_attempt_timeout.
-                If None, defaults to operation_timeout.
-            - retryable_errors: a list of errors that will be retried if encountered.
-                Only idempotent mutations will be retried. Defaults to the Table's
-                default_retryable_errors.
+        Args:
+          - row_key: the row to apply mutations to
+          - mutations: the set of mutations to apply to the row
+          - operation_timeout: the time budget for the entire operation, in seconds.
+              Failed requests will be retried within the budget.
+              Defaults to the Table's default_operation_timeout
+          - attempt_timeout: the time budget for an individual network request, in seconds.
+              If it takes longer than this time to complete, the request will be cancelled with
+              a DeadlineExceeded exception, and a retry will be attempted.
+              Defaults to the Table's default_attempt_timeout.
+              If None, defaults to operation_timeout.
+          - retryable_errors: a list of errors that will be retried if encountered.
+              Only idempotent mutations will be retried. Defaults to the Table's
+              default_retryable_errors.
         Raises:
-             - DeadlineExceeded: raised after operation timeout
-                 will be chained with a RetryExceptionGroup containing all
-                 GoogleAPIError exceptions from any retries that failed
-             - GoogleAPIError: raised on non-idempotent operations that cannot be
-                 safely retried.
-            - ValueError if invalid arguments are provided
+           - DeadlineExceeded: raised after operation timeout
+               will be chained with a RetryExceptionGroup containing all
+               GoogleAPIError exceptions from any retries that failed
+           - GoogleAPIError: raised on non-idempotent operations that cannot be
+               safely retried.
+          - ValueError if invalid arguments are provided
         """
         operation_timeout, attempt_timeout = _get_timeouts(
             operation_timeout, attempt_timeout, self

setup.py

@@ -37,8 +37,8 @@
 # 'Development Status :: 5 - Production/Stable'
 release_status = "Development Status :: 5 - Production/Stable"
 dependencies = [
-    "google-api-core[grpc] >= 2.16.0, <3.0.0dev,!=2.2.*,!=2.3.*,!=2.4.*,!=2.5.*,!=2.6.*,!=2.7.*,!=2.8.*,!=2.9.*,!=2.10.*",
-    "google-cloud-core >= 2.0.0, <3.0.0dev",
+    "google-api-core[grpc] >= 2.16.0, <3.0.0dev",
+    "google-cloud-core >= 1.4.4, <3.0.0dev",
     "grpc-google-iam-v1 >= 0.12.4, <1.0.0dev",
     "proto-plus >= 1.22.0, <2.0.0dev",
     "proto-plus >= 1.22.2, <2.0.0dev; python_version>='3.11'",

testing/constraints-3.7.txt

@@ -5,7 +5,7 @@
 #
 # e.g., if setup.py has "foo >= 1.14.0, < 2.0.0dev",
 # Then this file should have foo==1.14.0
-google-api-core==2.16.0rc0
+google-api-core==2.16.0
 google-cloud-core==2.0.0
 grpc-google-iam-v1==0.12.4
 proto-plus==1.22.0

testing/constraints-3.8.txt

@@ -5,7 +5,7 @@
 #
 # e.g., if setup.py has "foo >= 1.14.0, < 2.0.0dev",
 # Then this file should have foo==1.14.0
-google-api-core==2.16.0rc0
+google-api-core==2.16.0
 google-cloud-core==2.0.0
 grpc-google-iam-v1==0.12.4
 proto-plus==1.22.0