iverase commented on PR #14176: URL: https://github.com/apache/lucene/pull/14176#issuecomment-2629385255
These numbers look great1 I want to run this change on the geo benchmarks but I expect similar speed ups. I am planning to do it early next week. One thing I am unhappy with is with the introduction of another scratch array. I wonder if we can move the docIds array [here](https://github.com/apache/lucene/blob/dd76dc4973fdbf554f0416902ead560da00bdad6/lucene/core/src/java/org/apache/lucene/util/bkd/BKDReader.java#L1032) into the docIdswriter and avoid the introduction of this variable? -- 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]
