Page MenuHomeElementl

[execute_in_process improvements 2/n] method to retrieve output values from execution result.
AbandonedPublic

Authored by cdecarolis on Feb 8 2021, 7:33 PM.

Details

Reviewers
alangenfeld
Summary

This adds the requisite machinery to ExecutionResult in order to retrieve output values.

Test Plan

Added output checking to unit tests.

Diff Detail

Repository
R1 dagster
Branch
execution_result_output_values
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 8 2021, 7:50 PM
Harbormaster failed remote builds in B25420: Diff 31008!

lets take a closer look at https://dagster.phacility.com/D5834 to see if we should try a new approach here

python_modules/dagster/dagster/core/execution/execute.py
102–113

oh my - missed the diff that added this

nasty