The Rusoto Error variant is too specific to handle both get and put errors. I'm therefore breaking it out into two variants, RusotoGet and RusotoPut. This will help us handle errors related to the put_item Rusoto method.
Depends on D3960
Paths
| Differential D3961 Authored by varun on May 6 2022, 1:26 PM.
Details
Summary The Rusoto Error variant is too specific to handle both get and put errors. I'm therefore breaking it out into two variants, RusotoGet and RusotoPut. This will help us handle errors related to the put_item Rusoto method. Depends on D3960 Test Plan cargo build
Diff Detail
Event TimelineHerald added subscribers: • abosh, • adrian, ashoat. · View Herald TranscriptMay 6 2022, 1:26 PM2022-05-06 13:26:31 (UTC-7) Harbormaster completed remote builds in B8883: Diff 12386.May 6 2022, 1:33 PM2022-05-06 13:33:08 (UTC-7) This revision is now accepted and ready to land.May 9 2022, 3:25 AM2022-05-09 03:25:24 (UTC-7) Comment Actions I'm not sure why this is in an "accepted" state. I hope it cannot be landed... After I accept, rust diffs should not be accepted, I know rust far too little to be decisive. Adding Jim as blocking. This revision now requires review to proceed.May 9 2022, 4:53 AM2022-05-09 04:53:33 (UTC-7) This revision is now accepted and ready to land.May 9 2022, 10:08 AM2022-05-09 10:08:40 (UTC-7) Closed by commit rCOMM15980a381f92: [Identity] Add new Error variant (authored by varun). · Explain WhyMay 27 2022, 12:54 PM2022-05-27 12:54:35 (UTC-7) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 13184 services/identity/src/database.rs
|