-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat(pubsub/pstest)!: Add status code to error injection (#2926)
A status code is added into `errorInjectionReactor`, this is necessary as the retry setting for some calls depends on the error code. Also the `errorInjectionReactor` is not exported. Note that the previous version is same as having an `Unknown` code as the server will always wrap the error to a proper status error. BREAKING CHANGE: this will invalidate the previous exported ErrorInjectionReactor struct and WithErrorInjection function.
- Loading branch information
Zhongduo Lin (Jimmy)
committed
Sep 28, 2020
1 parent
9f44d66
commit b084dce
Showing
2 changed files
with
22 additions
and
11 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters