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

s192 - Better handle wrapped DatastoreExceptions #32

Merged
merged 1 commit into from
Jan 31, 2025
Merged

s192 - Better handle wrapped DatastoreExceptions #32

merged 1 commit into from
Jan 31, 2025

Conversation

jlorper
Copy link
Member

@jlorper jlorper commented Jan 31, 2025

Fixes

Change implications

  • breaking change to API? no
  • changes dependencies? no

@jlorper jlorper changed the title Look for DatastoreException in cause of any kind of Throwable. Tests s192 - Better handle wrapped DatastoreExceptions Jan 31, 2025
@jlorper jlorper requested a review from eschultink January 31, 2025 17:46
@jlorper jlorper self-assigned this Jan 31, 2025
@jlorper jlorper requested a review from parkcoll January 31, 2025 18:48
@jlorper jlorper merged commit 6808293 into main Jan 31, 2025
3 checks passed
@jlorper jlorper deleted the s192-better-ds-exception-retry branch January 31, 2025 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@parkcoll parkcoll parkcoll approved these changes

@eschultink eschultink Awaiting requested review from eschultink

Assignees

@jlorper jlorper

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jlorper @parkcoll