Should we have an `assert false` here? Behavior of `nextDoc` and `advance` are undefined when the iterator is exhausted so we shouldn't have anything that relies on the fact that these methods return `NO_MORE_DOCS` when the iterator is exhausted? 