Page MenuHomePhabricator

Remove executePlan dagster-graphql call
ClosedPublic

Authored by dgibson on Oct 19 2020, 8:43 PM.

Details

Summary

The only remaining callsites of this were executing a plan in-process and didn't need to be using dagster-graphql in the first place. Switch those callsites over to call execute_plan directly and then remove executePlan altogether from dagster-graphql.

Test Plan

BK

Diff Detail

Repository
R1 dagster
Branch
killtheoperator2
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

rm dagster-graphql from setup.py as well

Harbormaster returned this revision to the author for changes because remote builds failed.Oct 19 2020, 9:09 PM
Harbormaster failed remote builds in B19801: Diff 24024!

yeahfuckyeah

python_modules/dagster-graphql/dagster_graphql/client/mutations.py
0

can this not get dropped too?

This revision is now accepted and ready to land.Oct 20 2020, 4:39 PM

up, remove that class you mentinoed

This revision was automatically updated to reflect the committed changes.