We already do this check in the outer method. If we want to check here, maybe better as an invariant?

I'm probably missing something, because I see this is used in copy_required_intermediates_for_execution as well, but do we need to fetch events from the instance? Can we not compare step_keys_to_execute with the set of steps in the full plan or something?

if we phrased this as

if not asset_store or asset_store == mem_asset_store:
    return False

we could avoid the continue

Are we able to raise an informative error here? I think that "Cannot perform reexecution with non persistent intermediates manager" will likely confuse users in the asset store world.

switching to a simpler check

it's a bit tricky to raise the right error given we need to fallback to check intermediate storage