is there a reason why

with_executor

https://github.com/pantsbuild/pants/blob/master/src/rust/engine/src/externs/interface.rs#L1778` needs to take a

PyExecutor

and not a

&PyExecutor

?

does it break anything when you try to fix it?