msfroh commented on a change in pull request #2088:
URL: https://github.com/apache/lucene-solr/pull/2088#discussion_r527907408
##########
File path: lucene/CHANGES.txt
##########
@@ -184,6 +184,9 @@ Bug fixes
* LUCENE-9365: FuzzyQuery was missing matches when prefix length was equal to
the term length
(Mark Harwood, Mike Drob)
+* LUCENE-9617: Reset lowestUnassignedFieldNumber on FieldNumbers.clear(), to
avoid leaking
Review comment:
Thinking backwards from the user (who probably cares about the slow
memory leak more than anything), how about:
```
Fix per-field memory leak in IndexWriter.deleteAll(). Reset next available
internal field number to 0 on FieldInfos.clear(), to avoid wasting FieldInfo
references.
```
It gets the user-visible impact out front, explains the fix, and what was
leaking, all in about 20 words.
----------------------------------------------------------------
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.
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
