Page MenuHomeElementl
Differential D8393

Add instance to compute log manager
ClosedPublic
Actions

Authored by • johann on Jun 16 2021, 1:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 24, 12:30 AM
Unknown Object (File)
Wed, May 24, 12:30 AM
Unknown Object (File)
Thu, May 18, 2:22 AM
Unknown Object (File)
Sun, May 14, 8:26 AM
Unknown Object (File)
Sat, May 13, 11:11 PM
Unknown Object (File)
Apr 30 2023, 4:24 PM
Unknown Object (File)
Apr 29 2023, 6:43 PM
Unknown Object (File)
Apr 16 2023, 12:32 PM
View All 51 Files
Subscribers
None

Details

Reviewers
• prha
• dgibson
Commits
R1:6dbacf29da37: Add instance to compute log manager
Summary

Compute log managers should be able to access config on the instance

Test Plan

unit

Diff Detail

Repository
R1 dagster
Branch
add-instance-to-compute-logs (branched from master)
Lint
Lint Passed
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
python_modules/
dagster/
dagster/
core/
instance/
1 line
storage/
3 lines
dagster_tests/
core_tests/
storage_tests/
7 lines

Diff 39615

View Options

python_modules/dagster/dagster/core/instance/__init__.py

Loading...
View Options

python_modules/dagster/dagster/core/storage/compute_log_manager.py

Loading...
View Options

python_modules/dagster/dagster_tests/core_tests/storage_tests/test_compute_log_manager.py

Loading...