Differential D8843

have S3ComputeLogManager implement CapturedLogManager interface
Needs ReviewPublic
Actions

Authored by prha on Jul 12 2021, 9:58 PM.

Tags

None

Subscribers

None

Details

Reviewers

alangenfeld
sandyryza

Summary

Depends on D8385, D8840

Test Plan

bk

Diff Detail

Repository

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

prha created this revision.Jul 12 2021, 9:58 PM

Harbormaster completed remote builds in B33596: Diff 41473.Jul 12 2021, 10:42 PM

prha requested review of this revision.Jul 12 2021, 10:42 PM

rebase

Harbormaster failed remote builds in B33748: Diff 41671!Jul 15 2021, 5:17 PM

Revision Contents
Changeset List

Path

Size

python_modules/

libraries/

dagster-aws/

dagster_aws/

s3/

compute_log_manager.py

91 lines

dagster_aws_tests/

s3_tests/

test_compute_log_manager.py

12 lines

Diff 41671

python_modules/libraries/dagster-aws/dagster_aws/s3/compute_log_manager.py

Loading...

python_modules/libraries/dagster-aws/dagster_aws_tests/s3_tests/test_compute_log_manager.py

Loading...