Re: [PR] Add list initial capacity in FirstPassGroupingCollector#getTopGroups. [lucene]

2025-06-29 Thread via GitHub
vsop-479 commented on PR #14777: URL: https://github.com/apache/lucene/pull/14777#issuecomment-3017423555 > what motivated this change? Was it performance related or general code hygiene? Thanks @vigyasharma , Since we use grouping function (self-developed base on `FirstPassGroupingC

Re: [PR] Add list initial capacity in FirstPassGroupingCollector#getTopGroups. [lucene]

2025-06-28 Thread via GitHub
vigyasharma merged PR #14777: URL: https://github.com/apache/lucene/pull/14777 -- 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

Re: [PR] Add list initial capacity in FirstPassGroupingCollector#getTopGroups. [lucene]

2025-06-12 Thread via GitHub
github-actions[bot] commented on PR #14777: URL: https://github.com/apache/lucene/pull/14777#issuecomment-2968726519 This PR does not have an entry in lucene/CHANGES.txt. Consider adding one. If the PR doesn't need a changelog entry, then add the skip-changelog label to it and you will stop

[PR] Add list initial capacity in FirstPassGroupingCollector#getTopGroups. [lucene]

2025-06-12 Thread via GitHub
vsop-479 opened a new pull request, #14777: URL: https://github.com/apache/lucene/pull/14777 ### Description -- 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