shubhamvishu commented on PR #15982: URL: https://github.com/apache/lucene/pull/15982#issuecomment-4480347905
@iprithv Thanks for iterating here and looks ready. I think I understand now what was my confusion around double accounting(which we are not doing apparently). I added a small comment above to mention that in a inline code comment. Could you also run [luceneutil benchmarks](https://github.com/mikemccand/luceneutil/) with you PR to see how this impacts the vector indexing or merging etc? -- 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]
