Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: update the accounting of partial batch mutations #2149

Merged
merged 32 commits into from
Mar 27, 2024
Merged

fix: update the accounting of partial batch mutations #2149

Changes from 1 commit
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
7de3aba
Fixed retrying of partial errors
ron-gal Mar 6, 2024
aec1125
Minor changes
ron-gal Mar 6, 2024
e6485ee
fix tests
ron-gal Mar 6, 2024
e370774
fix tests
ron-gal Mar 7, 2024
f0f4e07
fix tests
ron-gal Mar 7, 2024
4c0f12f
fix tests
ron-gal Mar 7, 2024
bc486e5
fix callable order
ron-gal Mar 12, 2024
dcd6d15
Merge branch 'googleapis:main' into retry
ron-gal Mar 13, 2024
bcb98d7
Fixed PR comments
ron-gal Mar 14, 2024
9d225a3
reverted irrelevant changes
ron-gal Mar 14, 2024
64da251
reverted irrelevant changes
ron-gal Mar 14, 2024
7362b10
add tests
ron-gal Mar 14, 2024
3bf1b6f
add tests
ron-gal Mar 14, 2024
7c66089
add tests
ron-gal Mar 14, 2024
7515f29
add tests
ron-gal Mar 14, 2024
35d4c67
fix lint
ron-gal Mar 15, 2024
08bcd36
Some PR fixes
ron-gal Mar 19, 2024
3386f1d
more PR fixes
ron-gal Mar 19, 2024
ccb0a49
more PR fixes
ron-gal Mar 19, 2024
feb4985
fix lint
ron-gal Mar 19, 2024
3f6c60a
fix test
ron-gal Mar 20, 2024
d57e50f
fix lint
ron-gal Mar 20, 2024
820d877
add test
ron-gal Mar 21, 2024
6543543
add tests
ron-gal Mar 21, 2024
3f32cb5
add tests
ron-gal Mar 21, 2024
a7fa3dc
add tests
ron-gal Mar 21, 2024
13f9c92
remove LOCAL_STATUS
ron-gal Mar 21, 2024
2637e95
fix lint
ron-gal Mar 21, 2024
c263e51
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Mar 27, 2024
a0b6f88
fix clirr warning
ron-gal Mar 27, 2024
ae9b8c0
fix comment
ron-gal Mar 27, 2024
1570035
Merge branch 'googleapis:main' into retry
ron-gal Mar 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
add test
  • Loading branch information
@ron-gal
ron-gal committed Mar 21, 2024
commit 820d877bb1b63a65dc3cb8f96096ad3aa37599e9
7 changes: 6 additions & 1 deletion ...gtable/src/test/java/com/google/cloud/bigtable/data/v2/stub/EnhancedBigtableStubTest.java
View file
Open in desktop
Expand Up @@ -617,7 +617,10 @@ public void testBatchMutationsPartialFailure() {
.addEntries(
MutateRowsResponse.Entry.newBuilder()
.setIndex(1)
.setStatus(Status.newBuilder().setCode(Code.PERMISSION_DENIED_VALUE))
.setStatus(
Status.newBuilder()
.setCode(Code.PERMISSION_DENIED_VALUE)
.setMessage("fake partial error"))
.build())
.build());
observer.onCompleted();
Expand All @@ -630,6 +633,8 @@ public void testBatchMutationsPartialFailure() {
assertThat(batchingException.getMessage())
.contains(
"Batching finished with 1 partial failures. The 1 partial failures contained 1 entries that failed with: 1 ApiException(1 PERMISSION_DENIED).");
assertThat(batchingException.getMessage()).contains("fake partial error");
assertThat(batchingException.getMessage()).doesNotContain("INTERNAL");
}

@Test
Expand Down