HomePhabricator

Clean up up S3 resource (#2292)

Authored by nate on Sat, Mar 21, 5:41 PM.

Description

Clean up up S3 resource (#2292)

Summary:
Per #2292, resources like S3 shouldn't attempt to mirror underlying APIs, otherwise we'll always be playing catch-up with those of the underlying 3rd party library (not to mention handling multiple version compatibility).

This diff drops that behavior, and users will now need to access the session property on the S3 resource to access the underlying boto3 S3 resource.

Test Plan: unit

Reviewers: max, schrockn

Reviewed By: schrockn

Differential Revision: https://dagster.phacility.com/D2305

Details

Committed
nateSat, Mar 21, 8:37 PM
Reviewer
schrockn
Differential Revision
D2305: Clean up up S3 resource (#2292)
Parents
R1:4b0d44968966: Fix for AWS CLI (#2253)
Branches
Unknown
Tags
Unknown