Page MenuHomeElementl
Differential D5883

Remove experimental warning for queued run coordinator
ClosedPublic
Actions

Authored by • rexledesma on Jan 8 2021, 8:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 10, 5:04 PM
Unknown Object (File)
Wed, May 10, 7:25 AM
Unknown Object (File)
Wed, May 10, 1:33 AM
Unknown Object (File)
Apr 24 2023, 2:59 AM
Unknown Object (File)
Apr 7 2023, 9:00 PM
Unknown Object (File)
Apr 1 2023, 10:53 PM
Unknown Object (File)
Feb 21 2023, 10:04 PM
Unknown Object (File)
Jan 29 2023, 4:47 AM
View All 40 Files
Subscribers
None

Details

Reviewers
• johann
• dgibson
Commits
R1:716c5250bdd0: Remove experimental warning for queued run coordinator
Summary

As the title.

Test Plan

N/A

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• rexledesma created this revision.Jan 8 2021, 8:40 PM
• rexledesma published this revision for review.Jan 8 2021, 8:48 PM
Harbormaster failed remote builds in B23946: Diff 29110!Jan 8 2021, 8:56 PM
• johann added a comment.Jan 8 2021, 9:01 PM

I believe doing it on the init is convention bc something breaks when it's on the class, it would be great if we could clarify the message though.

This particular experimental is coming off anyways though, you could change this diff to just remove it

• rexledesma updated this revision to Diff 29126.Jan 8 2021, 9:05 PM

remove experimental warning

• dgibson accepted this revision.Jan 8 2021, 9:05 PM

excellent.

This revision is now accepted and ready to land.Jan 8 2021, 9:05 PM
• rexledesma retitled this revision from Fix experimental warning for queued run coordinator to Remove experimental warning for queued run coordinator.Jan 8 2021, 9:06 PM
• rexledesma edited the summary of this revision. (Show Details)
• rexledesma edited the test plan for this revision. (Show Details)
Harbormaster completed remote builds in B23960: Diff 29126.Jan 8 2021, 9:21 PM
Closed by commit R1:716c5250bdd0: Remove experimental warning for queued run coordinator (authored by • rexledesma). · Explain WhyJan 8 2021, 9:56 PM
This revision was automatically updated to reflect the committed changes.
• rexledesma added a commit: R1:716c5250bdd0: Remove experimental warning for queued run coordinator.

Revision Contents
Changeset List

PathSize
python_modules/
dagster/
dagster/
core/
run_coordinator/
2 lines

Diff 29135

View Options

python_modules/dagster/dagster/core/run_coordinator/queued_run_coordinator.py

Loading...