mirror of https://github.com/apache/lucene.git
Fix error in TieredMergePolicy (#1028)
Fix error in comparing between bytes of candidates and bytes of max merge. It's wrong to use candidateSize rather than currentCandidateBytes comparing with maxMergeBytes.
This commit is contained in:
parent
781edf442b
commit
9f80fea502
|
@ -170,6 +170,8 @@ Bug Fixes
|
||||||
|
|
||||||
* LUCENE-10623: Error implementation of docValueCount for SortingSortedSetDocValues (Lu Xugang)
|
* LUCENE-10623: Error implementation of docValueCount for SortingSortedSetDocValues (Lu Xugang)
|
||||||
|
|
||||||
|
* GITHUB#1028: Fix error in TieredMergePolicy (Lin Jian)
|
||||||
|
|
||||||
Other
|
Other
|
||||||
---------------------
|
---------------------
|
||||||
|
|
||||||
|
|
|
@ -878,7 +878,7 @@ public class TieredMergePolicy extends MergePolicy {
|
||||||
// segments or that create a segment close to the
|
// segments or that create a segment close to the
|
||||||
// maximum allowed segment sized are permitted
|
// maximum allowed segment sized are permitted
|
||||||
if (candidateSize > 1
|
if (candidateSize > 1
|
||||||
&& (forceMergeRunning == false || candidateSize > 0.7 * maxMergeBytes)) {
|
&& (forceMergeRunning == false || currentCandidateBytes > 0.7 * maxMergeBytes)) {
|
||||||
final OneMerge merge = new OneMerge(candidate);
|
final OneMerge merge = new OneMerge(candidate);
|
||||||
if (verbose(mergeContext)) {
|
if (verbose(mergeContext)) {
|
||||||
message("add merge=" + segString(mergeContext, merge.segments), mergeContext);
|
message("add merge=" + segString(mergeContext, merge.segments), mergeContext);
|
||||||
|
|
Loading…
Reference in New Issue