Instead of creating a new BytesRef on every call to `nextValue`, it should modify in-place the `scratch` member that it inherits from `BytesValues` (potentially resizing the array if it is too small).