Page MenuHomePhabricator

[blob-service] Minor AWS cleanups
ClosedPublic

Authored by bartek on Aug 26 2023, 12:23 AM.
Tags
None
Referenced Files
F2103238: D8961.id30352.diff
Tue, Jun 25, 2:42 AM
F2103237: D8961.id30645.diff
Tue, Jun 25, 2:42 AM
F2103236: D8961.id30349.diff
Tue, Jun 25, 2:42 AM
F2103196: D8961.id.diff
Tue, Jun 25, 2:41 AM
F2103156: D8961.diff
Tue, Jun 25, 2:33 AM
Unknown Object (File)
Mon, Jun 24, 7:09 AM
Unknown Object (File)
Fri, Jun 21, 2:23 AM
Unknown Object (File)
Thu, Jun 13, 4:19 PM
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
Lint Not Applicable
Unit
Tests Not Applicable