close input load user boundary before return
ClosedPublic
Actions

Authored by alangenfeld on Jan 13 2021, 6:33 PM.

Details

Reviewers

dgibson
sandyryza
yuhan

Commits

R1:3a81a2c6b350: close input load user boundary before return

Summary

coming from https://buildkite.com/dagster/dagster/builds/15941#cb204426-b352-4a79-97ce-328486c4d805

im not sure this will actually fix the issue but cant hurt

Test Plan

Diff Detail

Repository

R1 dagster

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

alangenfeld created this revision.Jan 13 2021, 6:33 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Jan 13 2021, 6:50 PM

Harbormaster failed remote builds in B24267: Diff 29525!

^ flake

yuhan accepted this revision.Jan 13 2021, 7:46 PM

This revision is now accepted and ready to land.Jan 13 2021, 7:46 PM

This revision was landed with ongoing or failed builds.Jan 13 2021, 8:11 PM

Closed by commit R1:3a81a2c6b350: close input load user boundary before return (authored by alangenfeld). · Explain Why

This revision was automatically updated to reflect the committed changes.

alangenfeld added a commit: R1:3a81a2c6b350: close input load user boundary before return.

Revision Contents
Changeset List

Path

Size

python_modules/

dagster/

core/

execution/

plan/

inputs.py

4 lines

Diff 29568

View Options

python_modules/dagster/dagster/core/execution/plan/inputs.py

close input load user boundary before returnClosedPublicActions