Page MenuHomePhabricator

[blob-service] Minor AWS cleanups
ClosedPublic

Authored by bartek on Aug 26 2023, 12:23 AM.
Tags
None
Referenced Files
F2135290: D8961.diff
Fri, Jun 28, 10:38 AM
Unknown Object (File)
Wed, Jun 26, 8:43 AM
Unknown Object (File)
Tue, Jun 25, 2:42 AM
Unknown Object (File)
Tue, Jun 25, 2:42 AM
Unknown Object (File)
Tue, Jun 25, 2:42 AM
Unknown Object (File)
Tue, Jun 25, 2:41 AM
Unknown Object (File)
Tue, Jun 25, 2:33 AM
Unknown Object (File)
Mon, Jun 24, 7:09 AM
Subscribers

Details

Summary

By the way of D8959, performed some minor cleanups in blob codebase.

  1. Recently noticed that aws_sdk_{dynamodb,s3}::Client already have Arc inside, so we don't need to wrap it with Arc again.
  1. Also deleted unnecessary aws_types dependency. The type we're using is already included in aws_client. > It probably wasn't in the past and that's why it was added, but now it's not needed anymore.

Plus some very minor fixes to fix Clippy warns.

Depends on D8959

Test Plan

Blob service compiles and runs

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage