benwtrent commented on issue #11963: URL: https://github.com/apache/lucene/issues/11963#issuecomment-1323680557
This is a naive question, but is there a design principle I am missing that makes `VectorValues<T>` out of line? I don't know how often folks will read the vector values directly instead of just searching them. But, transforming everything to float has a performance impact during ingest. Additionally, if the user is expecting bytes, they may have to transform their floats again when reading values, which is another performance overhead. So, I am against 'floats everywhere', but I realize that other things in Lucene have unified types. I would prefer 'bytes everywhere' instead if we were pushing for a single array kind. -- 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
