Should we perhaps use `getFieldNames().fullName()` instead (or `getFieldNames().name()`)? I think this will give a much more actionable message.

If we keep using `getFieldNames().indexName()`, then it might be easier to store that locally rather than calling it twice. (Same comment for `PagedBytesIndexFieldData`)