In getExecutorFind(), we see a performance overhead from repeatedly copying QueryPlannerParams objects variously through the stack. While the original calls use std::move() on the QueryPlannerParams objects, they're stack allocated, not heap allocated, so there's still a copy involved.

Converting these into references or heap allocating them with smart pointers will improve performance simply by reducing the number of copies being made.