Differential D8490

Remove reference to output_manager decorator
ClosedPublic
Actions

Authored by • jordansanders on Jun 22 2021, 6:02 PM.

Tags

None

Referenced Files

	F2987262: D8490.diff
	Fri, Jun 2, 4:45 PM

	Unknown Object (File)
	Thu, May 25, 1:10 PM

	Unknown Object (File)
	Mon, May 8, 3:51 PM

	Unknown Object (File)
	Apr 1 2023, 9:53 PM

	Unknown Object (File)
	Mar 31 2023, 3:54 AM

	Unknown Object (File)
	Mar 31 2023, 3:16 AM

	Unknown Object (File)
	Mar 31 2023, 3:07 AM

	Unknown Object (File)
	Mar 21 2023, 5:42 PM

View All 51 Files

Subscribers

None

Details

Reviewers

• sandyryza
alangenfeld

Commits

R1:c2a85bca973e: Remove reference to output_manager decorator

Summary

This decorator was removed with the 0.10.0 release when we transitioned
from using OutputManagers directly to using IOManagers:

https://github.com/dagster-io/dagster/commit/3397d9c1eda139c81501d1157a3f72f9e9a334dc

But the remaining docstring caused some confusion in Slack:

https://dagster.slack.com/archives/C01U954MEER/p1624380774044300

Test Plan

unit

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• jordansanders created this revision.Jun 22 2021, 6:02 PM

Harbormaster completed remote builds in B32491: Diff 40026.Jun 22 2021, 6:29 PM

• jordansanders requested review of this revision.Jun 22 2021, 6:29 PM

• jordansanders added inline comments.Jun 22 2021, 6:32 PM

python_modules/dagster/dagster/core/storage/output_manager.py
58	I wonder a) how much any of these docstrings remain useful and b) how much the OutputManager/InputManagers remain useful or if they should be folded into the IOManager base class.

given the inline comment ill let @sandyryza accept with any other insight on what should change around this

Did a user come across this? I didn't think we exposed OutputManager in our docs at all. I don't think we need it around anymore.

This revision is now accepted and ready to land.Jun 24 2021, 3:30 PM

In D8490#222107, @sandyryza wrote:

Did a user come across this? I didn't think we exposed OutputManager in our docs at all. I don't think we need it around anymore.

Yeah, a user asked about it https://dagster.slack.com/archives/C01U954MEER/p1624380774044300

Closed by commit R1:c2a85bca973e: Remove reference to output_manager decorator (authored by • jordansanders). · Explain WhyJun 30 2021, 7:31 PM

This revision was automatically updated to reflect the committed changes.

• jordansanders added a commit: R1:c2a85bca973e: Remove reference to output_manager decorator.

Revision Contents
Changeset List

Path

Size

python_modules/

dagster/

dagster/

core/

storage/

output_manager.py

2 lines

Diff 40577

python_modules/dagster/dagster/core/storage/output_manager.py

Loading...