Page MenuHomePhabricator

(dagster-cli-api-run-launcher-5) Make subset execution work over cli api
ClosedPublic

Authored by schrockn on Thu, May 21, 9:48 PM.

Details

Summary

Made a test for subset execution here and found a bug.

Depends on D3012

Test Plan

BK

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

schrockn created this revision.Thu, May 21, 9:48 PM
schrockn requested review of this revision.Thu, May 21, 10:16 PM
schrockn updated this revision to Diff 14518.Fri, May 22, 3:32 AM
schrockn added reviewers: alangenfeld, max.

up

max accepted this revision.Fri, May 22, 5:38 PM
max added inline comments.
python_modules/dagster/dagster/cli/api.py
163

what happens in this case

This revision is now accepted and ready to land.Fri, May 22, 5:38 PM
schrockn added inline comments.Fri, May 22, 6:38 PM
python_modules/dagster/dagster/cli/api.py
162–163

it logs an error in _get_recon_pipeline and does not execute