kaivalnp commented on PR #16932: URL: https://github.com/apache/lucene/pull/16932#issuecomment-6582412918
@CH-Abhinav consolidating the "raw vector alignment" variable into a single place is a good idea, but it's currently used in only two places -- I'm inclined to keep it as-is for now. Since this PR closely mimics the original change for a similar vector format, I'm hoping to merge this soon. Tagging contributors of the original change (@goankur @uschindler @gsmiller) in case they have feedback. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
