This does add some overhead to any execution paths where we invoke the dagster-graphql CLI,
e.g. Dask and Dockerized Airflow execution.
while we may not want to remove the sync manager due to costs - I do think we should definitely move forward with the blocking mutation call change
oh this is how we structured this ... hmmm maybe its too much to make the schema that different here
it would be nice to update this type to not return any "result" fields