In PlanExecutorImpl::getNext(), a result is skipped if the caller requests a record ID but none is provided on the WSM produced by the PlanStage tree.

This can result in some very confusing behavior for internal users of PlanExecutor, where documents are filtered out at the PlanExecutor layer. We should consider changing the behavior in this case to tassert() or log an error message instead of simply filtering out the document. If a caller is expecting a RecordID and the plan is not producing RecordIds, it indicates an issue in the way the plan was constructed.

This issue has no known external impact.