Skip to content
This repository was archived by the owner on Apr 1, 2026. It is now read-only.

Commit 89759fb

Browse files
committed
Added TODO + fixed system test
1 parent 115fed2 commit 89759fb

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

google/cloud/bigtable/data/exceptions.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,8 @@ def __repr__(self):
141141
return f"{self.__class__.__name__}({message!r}, {self.exceptions!r})"
142142

143143

144+
# TODO: When working on mutations batcher, rework exception handling to guarantee that
145+
# MutationsExceptionGroup only stores FailedMutationEntryErrors.
144146
class MutationsExceptionGroup(_BigtableExceptionGroup):
145147
"""
146148
Represents one or more exceptions that occur during a bulk mutation operation

tests/system/v2_client/test_data_api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -358,7 +358,7 @@ def test_table_mutate_rows_retries_timeout(data_table, rows_to_delete):
358358

359359
statuses = data_table.mutate_rows([row, row_2])
360360
assert statuses[0].code == Code.OK
361-
assert statuses[1].code == Code.INTERNAL
361+
assert statuses[1].code == Code.DEADLINE_EXCEEDED
362362

363363
# Retries with deadline 0 should do nothing.
364364
with mock.patch.object(

0 commit comments

Comments
 (0)