Page MenuHomeElementl

Move scheduler version check to optimize_for_dagit
ClosedPublic

Authored by dgibson on Mar 4 2021, 11:02 PM.

Details

Summary

This makes it so that every run worker / DagsterInstance.init() doesn't make a scheduler storage query on bootup.

Test Plan

BK

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Mar 4 2021, 11:20 PM
Harbormaster failed remote builds in B26886: Diff 32862!
alangenfeld added inline comments.
python_modules/dagster/dagster/core/instance/__init__.py
569–570

hmm should validate_stored_schedules just be in optimize_for_dagit ? i dunno

This revision is now accepted and ready to land.Mar 5 2021, 10:56 PM
python_modules/dagster/dagster/core/instance/__init__.py
366–368

oh nice