Fixes for dagster-mysql local runs
ClosedPublic
Actions

Authored by • dgibson on Jun 7 2021, 4:07 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Mar 29, 4:13 AM

	Unknown Object (File)
	Wed, Mar 22, 12:19 AM

	Unknown Object (File)
	Sat, Mar 18, 11:50 PM

	Unknown Object (File)
	Wed, Mar 15, 7:51 AM

	Unknown Object (File)
	Sat, Mar 11, 12:17 AM

	Unknown Object (File)
	Thu, Mar 9, 12:44 AM

	Unknown Object (File)
	Mon, Mar 6, 1:31 PM

	Unknown Object (File)
	Tue, Feb 28, 9:52 AM

View All 67 Files

Subscribers

None

Details

Reviewers

• sidkmenon
alangenfeld
• prha

Commits

R1:af575b062831: Fixes for dagster-mysql local runs

Summary

don't re-create the docker-compose file on each test, that is slow
add a new connection error to the set of errors we retry on

Test Plan

BK, run dagster-mysql locally

Diff Detail

Repository

R1 dagster

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• dgibson created this revision.Jun 7 2021, 4:07 PM

Harbormaster completed remote builds in B31758: Diff 39108.Jun 7 2021, 4:32 PM

• dgibson requested review of this revision.Jun 7 2021, 4:32 PM

lgtm! Haven't run into the Interface error personally but I figure if it's working locally for you it's fine!

This revision is now accepted and ready to land.Jun 7 2021, 4:33 PM

Closed by commit R1:af575b062831: Fixes for dagster-mysql local runs (authored by • dgibson). · Explain WhyJun 7 2021, 4:39 PM

This revision was automatically updated to reflect the committed changes.

• dgibson added a commit: R1:af575b062831: Fixes for dagster-mysql local runs.

Revision Contents
Changeset List

Path

Size

python_modules/

libraries/

dagster-mysql/

dagster_mysql/

utils.py

1 line

dagster_mysql_tests/

conftest.py

4 lines

Diff 39117

View Options

python_modules/libraries/dagster-mysql/dagster_mysql/utils.py