In PrimaryOnlyService::run we pass the stored shared_ptr<executor::ScopedTaskExector by value, which invokes a copy constructor and increases the refcount of the smart pointer needlessly. We should at least pass this value by const-ref.