[
https://issues.apache.org/jira/browse/LUCENE-9298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17080894#comment-17080894
]
ASF subversion and git services commented on LUCENE-9298:
---------------------------------------------------------
Commit 527e6516604e0f7ded72d8ac6f7b6b79884b62c5 in lucene-solr's branch
refs/heads/master from Nhat Nguyen
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=527e651 ]
LUCENE-9298: Fix TestBufferedUpdates
This test failed on Elastic CI because we did not add any term in the
loop. This commit ensures that we always add at least one docId, term
and query in the test.
> Improve RAM accounting in BufferedUpdates when deleted doc IDs and terms are
> cleared
> ------------------------------------------------------------------------------------
>
> Key: LUCENE-9298
> URL: https://issues.apache.org/jira/browse/LUCENE-9298
> Project: Lucene - Core
> Issue Type: Bug
> Affects Versions: 8.5
> Reporter: yubinglei
> Assignee: Simon Willnauer
> Priority: Minor
> Labels: easyfix, newbie, pull-request-available
> Fix For: master (9.0), 8.6
>
> Time Spent: 1h 40m
> Remaining Estimate: 0h
>
> As mentioned in this
> [https://github.com/apache/lucene-solr/pull/513#discussion_r399131681]
> the method clearDeletedDocIds in BufferedUpdates.java has a bug, it can't
> reset bytesUsed correctly
> {code:java}
> void clearDeletedDocIds() {
> deleteDocIDs.clear();
> bytesUsed.addAndGet(-deleteDocIDs.size() *
> BufferedUpdates.BYTES_PER_DEL_DOCID);
> }
> {code}
>
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
