PlanExecutor* exec is immediately dynamically casted to PlanExecutorPipeline* and checked to be non-nullptr anyway. In the places where this method is called the exec is already checked to be PlanExecutorPipeline, by changing the signature we could perform one less downcast.