Page MenuHomeElementl
Differential D6028

Add retries to check in dagster=postgres init that checks table names
ClosedPublic
Actions

Authored by • dgibson on Jan 15 2021, 3:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 16, 4:28 AM
Unknown Object (File)
Tue, Sep 13, 6:32 AM
Unknown Object (File)
Tue, Sep 13, 6:32 AM
Unknown Object (File)
Tue, Sep 13, 6:32 AM
Unknown Object (File)
Tue, Sep 13, 6:32 AM
Unknown Object (File)
Tue, Sep 13, 6:32 AM
Unknown Object (File)
Tue, Sep 13, 6:32 AM
Unknown Object (File)
Tue, Sep 13, 6:32 AM
View All 37 Files
Subscribers
None

Details

Reviewers
• prha
alangenfeld
• johann
• nate
Commits
R1:af02cdb364dc: Add retries to check in dagster=postgres init that checks table names
Summary

The extra logic we aded in dagster_postgres storage init to avoid calling create_all each time needs retries.

Test Plan

Start up the deploy-docker example from scratch (be sure to drop the postgres tables first so it's legitimately starting from scratch). No more postgres error during startup

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
python_modules/
libraries/
dagster-postgres/
dagster_postgres/
event_log/
4 lines
run_storage/
3 lines
schedule_storage/
4 lines

Diff 29824

View Options

python_modules/libraries/dagster-postgres/dagster_postgres/event_log/event_log.py

Loading...
View Options

python_modules/libraries/dagster-postgres/dagster_postgres/run_storage/run_storage.py

Loading...
View Options

python_modules/libraries/dagster-postgres/dagster_postgres/schedule_storage/schedule_storage.py

Loading...