From 722d4c576f481c9a0309d34bd20796c7c37afebd Mon Sep 17 00:00:00 2001 From: Mattie Fu Date: Tue, 11 Jul 2023 09:58:14 -0400 Subject: [PATCH] add the checks back --- .../google/cloud/bigtable/data/v2/models/BulkMutation.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/models/BulkMutation.java b/google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/models/BulkMutation.java index 7c66484ca3..e663018f41 100644 --- a/google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/models/BulkMutation.java +++ b/google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/models/BulkMutation.java @@ -89,7 +89,9 @@ public BulkMutation add(@Nonnull ByteString rowKey, @Nonnull Mutation mutation) long mutationCount = mutation.getMutations().size(); Preconditions.checkArgument( mutationCountSum + mutationCount <= MAX_MUTATION, - String.format("Too many mutations, got %s, limit is %s", mutationCountSum, MAX_MUTATION)); + String.format( + "Too many mutations, got %s, limit is %s", + mutationCountSum + mutationCount, MAX_MUTATION)); this.mutationCountSum += mutationCount; builder.addEntries( @@ -106,7 +108,6 @@ public BulkMutation add(@Nonnull ByteString rowKey, @Nonnull Mutation mutation) */ public BulkMutation add(@Nonnull RowMutationEntry entry) { Preconditions.checkNotNull(entry, "Row mutation entry can't be null"); - builder.addEntries(entry.toProto()); return this; }