HomeElementl

Initialize sub_process in execute_script_file (#4899)

Description

Initialize sub_process in execute_script_file (#4899)

Summary:
If the POpen constructor raises an exception for some reason, the variable is not initialized.

Test Plan: I tried for a while to come up with a situation where this would happen and was not successful, but https://dagster.slack.com/archives/C01U954MEER/p1632425935208000 suggests it is happening somehow some of the time.

Details

Provenance
dgibsonAuthored on Fri, Sep 24, 3:46 PM
GitHub <noreply@github.com>Committed on Fri, Sep 24, 3:46 PM
Parents
R1:58f9de5a77ed: Docker compose up a single service (#4904)
Branches
Unknown
Tags
Unknown