jainankitk commented on PR #14622: URL: https://github.com/apache/lucene/pull/14622#issuecomment-2867308579
Merging this change as it primarily adds new benchmark, and small bug fix with significant performance impact (more than 10x). Ran the unit tests locally for 10k iterations to confirm no regressions: ``` > Task :lucene:core:testClasses > Task :lucene:core:test > Task :lucene:core:wipeTaskTemp > Task :lucene:sandbox:test :lucene:sandbox:test (SUCCESS): 10000 test(s) > Task :lucene:sandbox:wipeTaskTemp The slowest suites (exceeding 1s) during this run: 172.96s TestHistogramCollectorManager (:lucene:sandbox) [Incubating] Problems report is available at: file:///Users/akjain/ws/lucene/build/reports/problems/problems-report.html BUILD SUCCESSFUL in 3m 17s 246 actionable tasks: 242 executed, 4 up-to-date ``` -- 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