Both `getNextDocument(...)` and `getNext(...)` inside `PlanExecutorSBE` checked that the executor is not disposed and also both called `checkFailPointPlanExecAlwaysFails(nss())`, before finally calling into `getNextImpl(...)`.
However, `getNextImpl(...)` performs these same checks again. As these checks were performed on the per-document hot path, it makes sense to only execute them once.