i think we should also consider renaming build_sub_pipeline to indicate idempotency/updated semantics.
@alangenfeld also want to make sure you are onboard with this
this is not strict enough. per discussion in slack it should be identical or fail
the solid_subset on the parent should always be None with the new invariant in place