iverase commented on PR #12600: URL: https://github.com/apache/lucene/pull/12600#issuecomment-1745118236
I am just trying to have an abstraction that can replace the BytesRef output for binary doc values with something that does not impose the internal representation of the bytes like BytesRef does. I hope this will avoid some extra byte copying when reading and when writing and avoid the risk of a humongous allocation when reading binary doc values. I am ok if we said that RandomaccessInput is not the right abstraction and we need something else. On the other hand it seems to fit well. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
