Page MenuHomePhabricator

[Identity] Add new Error variant
ClosedPublic

Authored by varun on May 6 2022, 1:26 PM.
Tags
None
Referenced Files
F3375380: D3961.id13184.diff
Tue, Nov 26, 7:57 PM
Unknown Object (File)
Sat, Nov 23, 2:33 PM
Unknown Object (File)
Sat, Nov 23, 10:09 AM
Unknown Object (File)
Wed, Nov 13, 10:49 AM
Unknown Object (File)
Thu, Nov 7, 4:18 PM
Unknown Object (File)
Tue, Nov 5, 7:11 PM
Unknown Object (File)
Sat, Nov 2, 9:25 AM
Unknown Object (File)
Oct 13 2024, 8:12 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

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun requested review of this revision.May 6 2022, 1:33 PM
This revision is now accepted and ready to land.May 9 2022, 3:25 AM

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 AM
This revision is now accepted and ready to land.May 9 2022, 10:08 AM
This revision was automatically updated to reflect the committed changes.